Author: tschoening
Date: Fri Oct  9 09:10:37 2015
New Revision: 1707678

URL: http://svn.apache.org/viewvc?rev=1707678&view=rev
Log:
LOGCXX-457: Removed hard coded 100 milliseconds to get a more uniform behaviour.

Modified:
    incubator/log4cxx/trunk/src/test/cpp/rolling/timebasedrollingtest.cpp

Modified: incubator/log4cxx/trunk/src/test/cpp/rolling/timebasedrollingtest.cpp
URL: 
http://svn.apache.org/viewvc/incubator/log4cxx/trunk/src/test/cpp/rolling/timebasedrollingtest.cpp?rev=1707678&r1=1707677&r2=1707678&view=diff
==============================================================================
--- incubator/log4cxx/trunk/src/test/cpp/rolling/timebasedrollingtest.cpp 
(original)
+++ incubator/log4cxx/trunk/src/test/cpp/rolling/timebasedrollingtest.cpp Fri 
Oct  9 09:10:37 2015
@@ -112,7 +112,7 @@ public:
       now += APR_USEC_PER_SEC;
     } }
 
-    delayUntilNextSecondWithMsg(100);
+    delayUntilNextSecondWithMsg();
 
     { for (int i = 0; i < 5; i++) {
         std::string message("Hello---");
@@ -156,7 +156,7 @@ public:
       now += APR_USEC_PER_SEC;
     } }
 
-    delayUntilNextSecond(100);
+    delayUntilNextSecond();
 
     { for (int i = 0; i <= 2; i++) {
         std::string message("Hello---");
@@ -226,7 +226,7 @@ public:
 
     filenames[3].resize(filenames[3].size() - 3);
 
-    delayUntilNextSecond(100);
+    delayUntilNextSecond();
 
     { for (int i = 0; i < 5; i++) {
         std::string message("Hello---");
@@ -273,7 +273,7 @@ public:
     } }
     filenames[3] = LOG4CXX_STR("output/test4.log");
 
-    delayUntilNextSecondWithMsg(100);
+    delayUntilNextSecondWithMsg();
 
     { for (int i = 0; i <= 2; i++) {
         std::string message("Hello---");
@@ -342,7 +342,7 @@ public:
     } }
     filenames[3] = LOG4CXX_STR("output/test5.log");
 
-    delayUntilNextSecondWithMsg(100);
+    delayUntilNextSecondWithMsg();
 
     { for (int i = 0; i < 5; i++) {
         std::string message("Hello---");
@@ -391,7 +391,7 @@ public:
 
     filenames[3] = LOG4CXX_STR("output/test6.log");
 
-    delayUntilNextSecond(100);
+    delayUntilNextSecond();
 
     { for (int i = 0; i < 5; i++) {
         std::string message("Hello---");
@@ -408,18 +408,18 @@ public:
 
   }
 
-  void delayUntilNextSecond(int millis) {
-    apr_time_t now = apr_time_now();
-    apr_time_t next = ((now / APR_USEC_PER_SEC) + 1) * APR_USEC_PER_SEC
-          + millis * 1000L;
+  void delayUntilNextSecond(size_t millis = 100)
+  {
+    apr_time_t now  = apr_time_now();
+    apr_time_t next = ((now / APR_USEC_PER_SEC) + 1) * APR_USEC_PER_SEC + 
millis * 1000L;
 
     apr_sleep(next - now);
   }
 
-  void delayUntilNextSecondWithMsg(int millis)
+  void delayUntilNextSecondWithMsg(size_t millis = 100)
   {
     std::cout << "Waiting until next second and 100 millis.";
-    delayUntilNextSecond(100);
+    delayUntilNextSecond(millis);
     std::cout << "Done waiting.";
   }
 };


Reply via email to