[
https://issues.apache.org/jira/browse/THRIFT-1317?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Dave Watson updated THRIFT-1317:
--------------------------------
Attachment:
0028-Remove-copy-constructibility-from-thrift-concurrency-2.patch
> Remove copy constructibility from
> -----------------------------------
>
> Key: THRIFT-1317
> URL: https://issues.apache.org/jira/browse/THRIFT-1317
> Project: Thrift
> Issue Type: Improvement
> Components: C++ - Library
> Reporter: Dave Watson
> Priority: Minor
> Attachments:
> 0028-Remove-copy-constructibility-from-thrift-concurrency-2.patch
>
>
> From 2af8bd64d2ec3a0792ab247ab7ffbc75b00ce302 Mon Sep 17 00:00:00 2001
> From: Jordan DeLong <[email protected]>
> 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(-)
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
