You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@thrift.apache.org by ro...@apache.org on 2011/10/03 21:37:09 UTC

svn commit: r1178532 - /thrift/trunk/lib/cpp/src/transport/TFileTransport.cpp

Author: roger
Date: Mon Oct  3 19:37:09 2011
New Revision: 1178532

URL: http://svn.apache.org/viewvc?rev=1178532&view=rev
Log:
THRIFT-1361 Optional replacement of pthread by boost::thread (minor fix)
FIX: member initialization order within constructor

Modified:
    thrift/trunk/lib/cpp/src/transport/TFileTransport.cpp

Modified: thrift/trunk/lib/cpp/src/transport/TFileTransport.cpp
URL: http://svn.apache.org/viewvc/thrift/trunk/lib/cpp/src/transport/TFileTransport.cpp?rev=1178532&r1=1178531&r2=1178532&view=diff
==============================================================================
--- thrift/trunk/lib/cpp/src/transport/TFileTransport.cpp (original)
+++ thrift/trunk/lib/cpp/src/transport/TFileTransport.cpp Mon Oct  3 19:37:09 2011
@@ -98,7 +98,10 @@ TFileTransport::TFileTransport(string pa
   , writerThreadId_(0)
   , dequeueBuffer_(NULL)
   , enqueueBuffer_(NULL)
+  , notFull_(&mutex_)
+  , notEmpty_(&mutex_)
   , closing_(false)
+  , flushed_(&mutex_)
   , forceFlush_(false)
   , filename_(path)
   , fd_(0)
@@ -107,9 +110,6 @@ TFileTransport::TFileTransport(string pa
   , lastBadChunk_(0)
   , numCorruptedEventsInChunk_(0)
   , readOnly_(readOnly)
-  , notFull_(&mutex_)
-  , notEmpty_(&mutex_)
-  , flushed_(&mutex_)
 {
   openLogFile();
 }