From c17fe6b918015166d145936414a411589f03255c Mon Sep 17 00:00:00 2001
From: David Reiss <dreiss@apache.org>
Date: Tue, 29 Apr 2008 00:29:43 +0000
Subject: [PATCH] THRIFT: Nonblcoking bug fix

calling decrement with -1 even though param is an integer. Just made it clean by adding a decrement function

reviewed by: mcslee
revertable
test plan: ran on a test server


git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@665669 13f79535-47bb-0310-9956-ffa450edef68
---
 lib/cpp/src/server/TNonblockingServer.h | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/lib/cpp/src/server/TNonblockingServer.h b/lib/cpp/src/server/TNonblockingServer.h
index 0c3c6d54..de2e43b9 100644
--- a/lib/cpp/src/server/TNonblockingServer.h
+++ b/lib/cpp/src/server/TNonblockingServer.h
@@ -158,8 +158,12 @@ class TNonblockingServer : public TServer {
     return eventBase_;
   }
 
-  void incrementNumConnections(size_t incr=1) {
-    numTConnections_ += incr;
+  void incrementNumConnections() {
+    ++numTConnections_;
+  }
+
+  void decrementNumConnections() {
+    --numTConnections_;
   }
 
   size_t getNumConnections() {
@@ -329,7 +333,7 @@ class TConnection {
   }
 
   ~TConnection() {
-    server_->incrementNumConnections(-1);
+    server_->decrementNumConnections();
   }
 
   // Initialize
-- 
2.17.1