GitHub user jeking3 opened a pull request:
https://github.com/apache/thrift/pull/981
THRIFT-3038: fix up some volatiles in cpp
I addressed the concerns in TFileTransport (forceFlush_) and in
TThreadPoolServer, so items #2, #4, #5 from the original issue description.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jeking3/thrift defect/THRIFT-3038
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/thrift/pull/981.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #981
----
commit cb4849abc4156d64532989344a4b2c2fa8be8148
Author: Jim King <jim.k...@simplivity.com>
Date: 2016-04-05T17:00:24Z
THRIFT-3038: fix a couple races and removed volatile per analysis
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
