default format_time performance improvement

example/example.cpp

@@ -33,7 +33,7 @@ int main(int argc, char* argv[])
 
     auto start = system_clock::now();
 
-    const unsigned int howmany = 5000000;
+    const unsigned int howmany = 15000000;
     for(unsigned int i = 0; i < howmany ; i++)
         my_logger.info() << "Hello logger " << i;
 

include/c11log/details/fast_oss.h

@@ -27,7 +27,7 @@ public:
         return _str;
     }
 
-    void clear()
+    void reset_str()
     {
         _str.clear();
     }
@@ -68,6 +68,10 @@ public:
     {
         return _dev.str_ref();
     }
+	void reset_str()
+	{
+		_dev.reset_str();
+	}
 
 private:
     str_devicebuf _dev;

include/c11log/details/line_logger.h

@@ -42,9 +42,7 @@ public:
         // The move ctor should only be called on start of logging line,
         // where no logging happened yet for this line so no need to copy the string from the other
         _oss(),
-        _level(other._level)
+        _level(other._level) {};
-    {
-    };
 
 
     ~line_logger()

include/c11log/formatter.h

@@ -6,9 +6,11 @@
 #include <sstream>
 #include <iomanip>
 #include <thread>
+#include <cstdlib>
 
 #include "common_types.h"
 #include "details/os.h"
+#include "details/fast_oss.h"
 
 namespace c11log
 {
@@ -35,7 +37,7 @@ public:
     {
         _format_time(tp, dest);
         if(!logger_name.empty())
-            dest << " [" <<  logger_name << ":" << c11log::level::to_str(level) << "] ";
+            dest << " [" <<  logger_name << ':' << c11log::level::to_str(level) << "] ";
         else
             dest << " [" << c11log::level::to_str(level) << "] ";
 
@@ -48,35 +50,44 @@ private:
 } //namespace c11log
 
 
-
+// Format datetime like this: [2014-03-14 17:15:22]
 inline void c11log::formatters::default_formatter::_format_time(const log_clock::time_point& tp, std::ostream &dest)
 {
-
-#ifdef _MSC_VER
-    __declspec(thread) static std::tm last_tm = { 0, 0, 0, 0, 0, 0, 0, 0, 0};
-    __declspec(thread) static char last_time_str[64];
-#else
-    thread_local static std::tm last_tm = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0};
-    thread_local static char last_time_str[64];
-#endif
-
-    auto tm_now = details::os::localtime(log_clock::to_time_t(tp));
 	using namespace c11log::details::os;
-    if(last_tm != tm_now)
+	using namespace std::chrono;
-    {
+
 #ifdef _MSC_VER
-        ::sprintf_s
+    __declspec(thread) static std::tm s_last_tm = { 0, 0, 0, 0, 0, 0, 0, 0, 0};
+    __declspec(thread) static details::fast_oss s_time_oss;
 #else
-        ::snprintf
+
+	thread_local static details::fast_oss s_time_oss;
+	thread_local static std::time_t s_cache_time_t = 0;
 #endif
-        (last_time_str, sizeof(last_time_str), "[%d-%02d-%02d %02d:%02d:%02d]",
+
-         tm_now.tm_year + 1900,
+
-         tm_now.tm_mon + 1,
+	std::time_t tp_time_t = log_clock::to_time_t(tp);
-         tm_now.tm_mday,
+
-         tm_now.tm_hour,
+
-         tm_now.tm_min,
+	//Cache every second
-         tm_now.tm_sec);
+	if(tp_time_t != s_cache_time_t)
-        last_tm = tm_now;
+    {		
+		auto tm_now = details::os::localtime(tp_time_t);
+		s_time_oss.reset_str();
+		s_time_oss.fill('0');
+		s_time_oss << '[' << tm_now.tm_year + 1900 << '-';
+		s_time_oss.width(2);
+		s_time_oss << tm_now.tm_mon + 1 << '-';
+		s_time_oss.width(2);
+		s_time_oss << tm_now.tm_mday << ' ';
+		s_time_oss.width(2);
+		s_time_oss << tm_now.tm_hour << ':';
+		s_time_oss.width(2);
+		s_time_oss << tm_now.tm_min << ':';
+		s_time_oss.width(2);
+		s_time_oss << tm_now.tm_sec << ']';		
+		s_cache_time_t = tp_time_t;
     }
-    dest << last_time_str;
+	const std::string &s = s_time_oss.str_ref();
+	dest.write(s.c_str(), s.size());
 }