.

wcfs/client/wcfs.cpp

@@ -183,7 +183,7 @@ using std::max;
 using std::vector;
 
 
-#define TRACE 1
+#define TRACE 0
 #if TRACE
 #  define trace(format, ...) log::Debugf(format, ##__VA_ARGS__)
 #else
@@ -792,7 +792,17 @@ retry:
     if (f->_openErr != nil)
         return make_pair(nil, E(f->_openErr));
 
-    // XXX recheck the wconn was not closed while the open was in progress
+    // NOTE no need to recheck that wconn was not closed while the open was in
+    // progress: we'll return "success" but Conn.close will close the fileh.
+    // However it is indistinguishable from the following scenario:
+    //
+    //      T1                  T2
+    //
+    //  f = wconn.open()
+    //  # completes ok
+    //                      wconn.close()
+    //
+    //  # use f -> error
 
     retok = true;
     return make_pair(f, nil);