SERVER-4740 Don't use unsigned for time deltas

Leads to weird bugs when time goes backwards.

This commit doesn't resolve SERVER-4740 (Use monotonic clock sources for
Timer) but lessens the impact of using a non-monotonic clock.

1 files changed, 4 insertions, 4 deletions

diff --git a/src/mongo/db/clientcursor.h b/src/mongo/db/clientcursor.h
index ca8aa95ca23..ae6640fa39f 100644
--- a/src/mongo/db/clientcursor.h
+++ b/src/mongo/db/clientcursor.h
@@ -89,9 +89,9 @@ namespace mongo {
          * @param millis amount of idle passed time since last call
          * note called outside of locks (other than ccmutex) so care must be exercised
          */
-        bool shouldTimeout( unsigned millis );
-        void setIdleTime( unsigned millis );
-        unsigned idleTime() const { return _idleAgeMillis; }
+        bool shouldTimeout( int millis );
+        void setIdleTime( int millis );
+        int idleTime() const { return _idleAgeMillis; }
 
         uint64_t getLeftoverMaxTimeMicros() const { return _leftoverMaxTimeMicros; }
         void setLeftoverMaxTimeMicros( uint64_t leftoverMaxTimeMicros ) {
@@ -184,7 +184,7 @@ namespace mongo {
         OpTime _slaveReadTill;
 
         // How long has the cursor been idle?
-        unsigned _idleAgeMillis;
+        int _idleAgeMillis;
 
         // TODO: Document.
         uint64_t _leftoverMaxTimeMicros;