SERVER-4328 cleaning of d_concurrency for readability. removed the 't' state for temprelease. that was making the code complicated it wasn't work it.

we can track that in LockState if we like, which would be good for assertion puproses, but this is better than the way it was.

1 files changed, 5 insertions, 2 deletions

diff --git a/src/mongo/db/d_concurrency.h b/src/mongo/db/d_concurrency.h
index d2cd75563c3..d26211a961b 100644
--- a/src/mongo/db/d_concurrency.h
+++ b/src/mongo/db/d_concurrency.h
@@ -32,9 +32,12 @@ namespace mongo {
             int local;
         };
         class GlobalWrite : boost::noncopyable { // recursive is ok
-            const bool stopGreed;
-            const char old;
+            const bool stoppedGreed;
         public:
+            /** @param stopGreed after acquisition stop greediness of other threads for write locks. this should generally not 
+                                 be used it is for exceptional circumstances. journaling uses it. perhaps this should go away, 
+                                 it makes the software more complicated.
+            */
             GlobalWrite(bool stopGreed = false); 
             ~GlobalWrite();
             void downgrade(); // W -> R