You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2012/06/19 23:44:42 UTC
[jira] [Commented] (THRIFT-1626) concurrency::Mutex timedlock fix
and lesser improvements
[ https://issues.apache.org/jira/browse/THRIFT-1626?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13397092#comment-13397092 ]
Hudson commented on THRIFT-1626:
--------------------------------
Integrated in Thrift #503 (See [https://builds.apache.org/job/Thrift/503/])
THRIFT-1626 concurrency::Mutex timedlock fix and lesser improvements
Patch: Andrew Majorov (Revision 1351845)
Result = SUCCESS
roger : http://svn.apache.org/viewvc/?view=rev&rev=1351845
Files :
* /thrift/trunk/lib/cpp/src/thrift/concurrency/Mutex.cpp
* /thrift/trunk/lib/cpp/src/thrift/protocol/TProtocol.h
* /thrift/trunk/lib/cpp/src/thrift/transport/TSocket.h
* /thrift/trunk/lib/cpp/src/thrift/transport/TTransportException.cpp
> concurrency::Mutex timedlock fix and lesser improvements
> --------------------------------------------------------
>
> Key: THRIFT-1626
> URL: https://issues.apache.org/jira/browse/THRIFT-1626
> Project: Thrift
> Issue Type: Bug
> Components: C++ - Library
> Affects Versions: 0.8, 0.9
> Reporter: Andrew Majorov
> Labels: patch
> Attachments: thrift_timedlock_fix_improvements.patch
>
>
> Thrift concurrency::Mutex have had its timedlock implemented slightly wrong.
> Patched it. Added a pretty simple emulation when a platform does not provide pthread_mutex_timedlock. The solution seems to be close to the one used in Mono for Android. Manually tested under android-8, works fine.
> Patch summary:
> - Correct timed lock timeouts and emulation on platforms where it is not supported.
> - Header config.h now being included conditionally as it supposed to.
> - Make internally used preprocessor definition names more specific and more likely to be unique.
> This will suppress error messages and wrong endianness deduction when compiling Thrift core C++
> library on Android.
> - Reorder header inclusion to ensure config.h will always be included before any platform stuff.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira