moved to log_msg

include/c11log/details/line_logger.h

@@ -2,7 +2,7 @@
 
 #include "../common_types.h"
 #include "../logger.h"
-#include "fast_oss.h"
+#include "stack_oss.h"
 
 
 // line logger class. should be used by the logger as an rvalue only.
@@ -20,16 +20,19 @@ class line_logger
 public:
     line_logger(logger* callback_logger, level::level_enum msg_level, bool enabled):
         _callback_logger(callback_logger),
+        _log_msg(),
         _oss(),
-        _level(msg_level),
         _enabled(enabled)
     {
         if(enabled)
         {
+            _log_msg.when = log_clock::now();
+            _log_msg.msg_level = msg_level;
             callback_logger->_formatter->format_header(callback_logger->_logger_name,
-                    msg_level,
-                    log_clock::now(),
+                    _log_msg.msg_level,
+                    _log_msg.when,
                     _oss);
+            _log_msg.header_size = _oss.size();
         }
     }
 
@@ -38,12 +41,12 @@ public:
     line_logger& operator=(const line_logger&) = delete;
     line_logger& operator=(line_logger&&) = delete;
 
-    // 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
     line_logger(line_logger&& other) :
         _callback_logger(other._callback_logger),
+        _log_msg(other._log_msg),
+        // 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 oss from the other
         _oss(),
-        _level(other._level),
         _enabled(other._enabled) {}
 
 
@@ -52,26 +55,28 @@ public:
         if (_enabled)
         {
             _oss << os::eol();
-            _callback_logger->_log_it(_oss.buf(), _level);
+            _log_msg.msg_buf = _oss.buf();
+            _callback_logger->_log_it(_log_msg);
         }
     }
 
 
     template<typename T>
-    line_logger&& operator<<(const T& msg)
+    line_logger&& operator<<(const T& what)
     {
         if (_enabled)
-            _oss << msg;
+            _oss << what;
 
         return std::move(*this);
     }
 
 private:
     logger* _callback_logger;
-    details::fast_oss _oss;
-    level::level_enum _level;
+    c11log::log_msg _log_msg;
+    details::stack_oss _oss;
     bool _enabled;
 
+
 };
 } //Namespace details
 } // Namespace c11log

include/c11log/details/stack_buf.h

@@ -15,18 +15,18 @@ namespace details
 {
 
 template<std::size_t STACK_SIZE=128>
-class fast_buf
+class stack_buf
 {
 public:
-    fast_buf():_stack_size(0) {}
-    ~fast_buf() {};
+    stack_buf():_stack_size(0) {}
+    ~stack_buf() {};
 
-    fast_buf(const bufpair_t& buf_to_copy):fast_buf()
+    stack_buf(const bufpair_t& buf_to_copy):stack_buf()
     {
         append(buf_to_copy);
     }
 
-    fast_buf(const fast_buf& other)
+    stack_buf(const stack_buf& other)
     {
         _stack_size = other._stack_size;
         if(!other._v.empty())
@@ -35,7 +35,7 @@ public:
             std::copy(other._stack_buf.begin(), other._stack_buf.begin()+_stack_size, _stack_buf.begin());
     }
 
-    fast_buf(fast_buf&& other)
+    stack_buf(stack_buf&& other)
     {
         _stack_size = other._stack_size;
         if(!other._v.empty())
@@ -45,31 +45,31 @@ public:
         other.clear();
     }
 
-    fast_buf& operator=(const fast_buf& other) = delete;
-    fast_buf& operator=(fast_buf&& other) = delete;
+    stack_buf& operator=(const stack_buf& other) = delete;
+    stack_buf& operator=(stack_buf&& other) = delete;
 
-    void append(const char* buf, std::size_t size)
+    void append(const char* buf, std::size_t buf_size)
     {
         //If we are aleady using _v, forget about the stack
         if(!_v.empty())
         {
-            _v.insert(_v.end(), buf, buf+ size);
+            _v.insert(_v.end(), buf, buf+ buf_size);
         }
         //Try use the stack
         else
         {
-            if(_stack_size+size <= STACK_SIZE)
+            if(_stack_size+buf_size <= STACK_SIZE)
             {
-                std::memcpy(&_stack_buf[_stack_size], buf, size);
-                _stack_size+=size;
+                std::memcpy(&_stack_buf[_stack_size], buf, buf_size);
+                _stack_size+=buf_size;
             }
             //Not enough stack space. Copy all to _v
             else
             {
-                _v.reserve(_stack_size+size);
+                _v.reserve(_stack_size+buf_size);
                 if(_stack_size)
                     _v.insert(_v.end(), _stack_buf.begin(), _stack_buf.begin() +_stack_size);
-                _v.insert(_v.end(), buf, buf+size);
+                _v.insert(_v.end(), buf, buf+buf_size);
             }
         }
     }
@@ -93,6 +93,14 @@ public:
             return bufpair_t(_stack_buf.data(), _stack_size);
     }
 
+    std::size_t size()
+    {
+        if(!_v.empty())
+            return _v.size();
+        else
+            return _stack_size;
+    }
+
 private:
     std::vector<char> _v;
     std::array<char, STACK_SIZE> _stack_buf;

include/c11log/details/stack_oss.h

@@ -2,7 +2,7 @@
 
 // Faster than ostringstream--returns its string by ref
 #include <ostream>
-#include "c11log/details/fast_buf.h"
+#include "c11log/details/stack_buf.h"
 
 namespace c11log
 {
@@ -12,7 +12,7 @@ namespace details
 class str_devicebuf:public std::streambuf
 {
 public:
-	using Base = std::streambuf;
+    using Base = std::streambuf;
     str_devicebuf() = default;
     ~str_devicebuf() = default;
 
@@ -23,24 +23,29 @@ public:
 
     bufpair_t buf()
     {
-        return _fastbuf.get();
+        return _stackbuf.get();
+    }
+
+    std::size_t size()
+    {
+        return _stackbuf.size();
     }
 
     void clear()
     {
-        _fastbuf.clear();
+        _stackbuf.clear();
     }
 
 protected:
     // copy the give buffer into the accumulated fast buffer
     std::streamsize xsputn(const char_type* s, std::streamsize count) override
     {
-        _fastbuf.append(s, static_cast<unsigned int>(count));
+        _stackbuf.append(s, static_cast<unsigned int>(count));
         return count;
     }
 
     int_type overflow(int_type ch) override
-    {        
+    {
         if (traits_type::not_eof(ch))
         {
             char c = traits_type::to_char_type(ch);
@@ -49,24 +54,29 @@ protected:
         return ch;
     }
 private:
-    fast_buf<192> _fastbuf;
+    stack_buf<192> _stackbuf;
 };
 
-class fast_oss:public std::ostream
+class stack_oss:public std::ostream
 {
 public:
-    fast_oss():std::ostream(&_dev) {}
-    ~fast_oss() = default;
+    stack_oss():std::ostream(&_dev) {}
+    ~stack_oss() = default;
 
-    fast_oss(const fast_oss& other) = delete;
-    fast_oss(fast_oss&& other) = delete;
-    fast_oss& operator=(const fast_oss& other) = delete;
+    stack_oss(const stack_oss& other) = delete;
+    stack_oss(stack_oss&& other) = delete;
+    stack_oss& operator=(const stack_oss& other) = delete;
 
     bufpair_t buf()
     {
         return _dev.buf();
     }
 
+    std::size_t size()
+    {
+        return _dev.size();
+    }
+
     void clear()
     {
         _dev.clear();