Reorder Logger() constructor args

Currently the order varies across the struct and constructor. As a nice
reminder GCC will throw a lovely warning at us. Fix the ordering.

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>

src/logging.cpp

@@ -150,9 +150,9 @@ void logging(){
 }
 
 Logger::Logger(overlay_params* in_params)
-  : m_logging_on(false),
-    m_values_valid(false),
-    m_params(in_params)
+  : m_params(in_params),
+    m_logging_on(false),
+    m_values_valid(false)
 {
   m_log_end = Clock::now() - 15s;
   SPDLOG_DEBUG("Logger constructed!");