SERVER-17817 Make ServiceContext create and manage Client objects.

Also, deduplicate Client::* method implementations, guard the identity of the
current CurOp of a Client with the Client's _mutex instead of the mutex guarding
the list of all clients.

Makes the currentClient object private to client.cpp, and all access to the
thread-bound client is now done with haveClient() and cc() free functions in the
mongo namespace.

Removes the vesitgal Client::shutdown() methods.

1 files changed, 0 insertions, 2 deletions

diff --git a/src/mongo/db/clientcursor.cpp b/src/mongo/db/clientcursor.cpp
index d0919fce1ab..b26fe10a583 100644
--- a/src/mongo/db/clientcursor.cpp
+++ b/src/mongo/db/clientcursor.cpp
@@ -290,7 +290,6 @@ namespace mongo {
 
         void run() {
             Client::initThread("clientcursormon");
-            Client& client = cc();
             Timer t;
             const int Secs = 4;
             while (!inShutdown()) {
@@ -299,7 +298,6 @@ namespace mongo {
                     CursorManager::timeoutCursorsGlobal(&txn, t.millisReset()));
                 sleepsecs(Secs);
             }
-            client.shutdown();
         }
     };