[prev in list] [next in list] [prev in thread] [next in thread] 

List:       helix-server-cvs
Subject:    [Server-cvs] engine/inputsource cachingsrcfinder.cpp,1.2,1.2.212.1
From:       ryuan () helixcommunity ! org
Date:       2012-11-27 3:31:17
[Download RAW message or body]

Update of /cvsroot/server/engine/inputsource
In directory cvs01.internal.helixcommunity.org:/tmp/cvs-serv660/server/engine/inputsource


Modified Files:
      Tag: SERVER_15_0
	cachingsrcfinder.cpp 
Log Message:
Synopsis: 
===============
Calling gettimeofday directly instead of calling the cached variable.

Branch: SERVER_15_0_RN, HEAD

Suggested Reviewers: Anyone

Description: 
===============
In the past, for performance consideration, when calling gettimeofday, we returned a \
calculated time instead of calling gettimeofday of C library. But in the multi-thread \
cases, there is a potential bug. So don't use the cached variable. Add two changes \
which are missed in yesterday's CR.


Files affected:
===============
server/engine/inputsource/cachingsrcfinder.cpp
server/engine/context/servsked.cpp

Testing Performed: 
===============
Integration Tests:
N/A

Performance Tests: 
N/A

QA Hints
========
None.



Index: cachingsrcfinder.cpp
===================================================================
RCS file: /cvsroot/server/engine/inputsource/cachingsrcfinder.cpp,v
retrieving revision 1.2
retrieving revision 1.2.212.1
diff -u -d -r1.2 -r1.2.212.1
--- cachingsrcfinder.cpp	10 May 2007 18:44:13 -0000	1.2
+++ cachingsrcfinder.cpp	27 Nov 2012 03:31:08 -0000	1.2.212.1
@@ -67,11 +67,7 @@
 
 // See GETTIMEOFDAY in server_engine.cpp
 
-#ifdef _UNIX
-#define GETTIME(t) (*g_bITimerAvailable) ? t = g_pNow->tv_sec : t = time(NULL)
-#else
 #define GETTIME(t) t = time(NULL);
-#endif
 
 // static initializations
 


_______________________________________________
Server-cvs mailing list
Server-cvs@helixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/server-cvs


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic