[mapguide-commits] r4286 - trunk/MgDev/Server/src/Common/Manager

svn_mapguide at osgeo.org svn_mapguide at osgeo.org
Tue Oct 6 16:37:14 EDT 2009


Author: brucedechant
Date: 2009-10-06 16:37:14 -0400 (Tue, 06 Oct 2009)
New Revision: 4286

Modified:
   trunk/MgDev/Server/src/Common/Manager/ServerManager.cpp
Log:
Add conditional compiler setting around ACE reactor connection limit error message as it only applies to Windows.

Modified: trunk/MgDev/Server/src/Common/Manager/ServerManager.cpp
===================================================================
--- trunk/MgDev/Server/src/Common/Manager/ServerManager.cpp	2009-10-06 20:35:33 UTC (rev 4285)
+++ trunk/MgDev/Server/src/Common/Manager/ServerManager.cpp	2009-10-06 20:37:14 UTC (rev 4286)
@@ -814,10 +814,11 @@
     // Increment the total connections
     m_totalConnections++;
 
-    // The limit for the current ACE reactor is 62.
+#ifdef _WIN32
+    // The limit for the current Windows ACE reactor is 62.
     // The only reason the value 55 is chosen is because it is slightly under this value and
     // so we can log an error as the # of active connections approaches the current ACE reactor limit.
-    // TODO: This error logging will need to be revisited if a different ACE reactor is used.
+    // TODO: This error logging will need to be revisited if a different ACE reactor is used on Windows.
     if(m_totalActiveConnections.value() > 55)
     {
         STRING strActiveConnections = L"";
@@ -829,6 +830,9 @@
         // We are dangerously close to exceeding the safe # of active connections for the current ACE reactor
         MG_LOG_ERROR_ENTRY(message.c_str());
     }
+#else
+    // Linux uses a different ACE reactor and so this error message doesn't apply.
+#endif
 }
 
 void MgServerManager::DecrementActiveConnections()



More information about the mapguide-commits mailing list