[THRIFT-1317] Remove copy constructibility from

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.8
• Component/s: C++ - Library
• Assignee: Dave Watson
• Reporter: Dave Watson
• Created: 31/Aug/11 17:27
• Updated: 01/Sep/11 17:41
• Resolved: 01/Sep/11 16:53

Description

From 2af8bd64d2ec3a0792ab247ab7ffbc75b00ce302 Mon Sep 17 00:00:00 2001
From: Jordan DeLong <[e-mail]>
Date: Tue, 20 Apr 2010 19:21:45 +0000
Subject: [PATCH 28/33] Remove copy constructibility from
thrift::concurrency::Guard

Summary:
Copy constructing this class currently will cause a mutex to
get unlocked twice. It looks like this was an unintended consequence of
changing the mutex_ member from a reference to a pointer.

Test Plan:
Compiled (random project using this). But anything
this change breaks probably has a copy construction bug in their guard
usage.

Revert Plan:
OK

—
lib/cpp/src/concurrency/Mutex.h | 3 ++-
1 files changed, 2 insertions, 1 deletions