Raise default log level to INFO, disable less important messages in builds without debug logging

Author: tstenner

src/api_config.cpp

@@ -76,7 +76,10 @@ void api_config::load_from_file(const std::string &filename) {
 		INI pt;
 		if (!filename.empty()) {
 			std::ifstream infile(filename);
-			if (infile.good()) pt.load(infile);
+			if (infile.good()) {
+				LOG_F(INFO, "Loading configuration from %s", filename.c_str());
+				pt.load(infile);
+			}
 		}
 
 		// read out the [ports] parameters
@@ -206,7 +209,7 @@ void api_config::load_from_file(const std::string &filename) {
 		force_default_timestamps_ = pt.get("tuning.ForceDefaultTimestamps", false);
 
 		// read the [log] settings
-		int log_level = pt.get("log.level", -1);
+		int log_level = pt.get("log.level", (int) loguru::Verbosity_INFO);
 		if (log_level < -3 || log_level > 9)
 			throw std::runtime_error("Invalid log.level (valid range: -3 to 9");
 

src/common.cpp

@@ -41,7 +41,7 @@ void lsl::ensure_lsl_initialized() {
 	if (!is_initialized) {
 		is_initialized = true;
 
-		loguru::g_stderr_verbosity = loguru::Verbosity_WARNING;
+		loguru::g_stderr_verbosity = loguru::Verbosity_INFO;
 #ifdef LOGURU_DEBUG_LOGGING
 		// Initialize loguru, mainly to print stacktraces on segmentation faults
 		int argc = 1;

src/tcp_server.cpp

@@ -329,7 +329,7 @@ void client_session::handle_read_query_outcome(error_code err) {
 						/* keep the client_session alive until the shortinfo is sent completely*/
 					});
 			} else
-				LOG_F(INFO, "%p got a shortinfo query response for the wrong query", this);
+				DLOG_F(INFO, "%p got a shortinfo query response for the wrong query", this);
 		}
 	} catch (std::exception &e) {
 		LOG_F(WARNING, "Unexpected error while parsing a client request: %s", e.what());
@@ -356,7 +356,7 @@ void client_session::handle_read_feedparams(
 				send_status_message(
 					"LSL/" + std::to_string(api_config::get_instance()->use_protocol_version()) +
 					" 505 Version not supported");
-				LOG_F(INFO, "%p Got a request for a too new protocol version", this);
+				DLOG_F(INFO, "%p Got a request for a too new protocol version", this);
 				return;
 			}
 			if (!request_uid.empty() && request_uid != serv_->info_->uid()) {

src/udp_server.cpp

@@ -136,7 +136,7 @@ void udp_server::handle_receive_outcome(error_code err, std::size_t len) {
 							});
 						return;
 					} else {
-						LOG_F(INFO, "%p Got shortinfo query for mismatching query string: %s", this,
+						DLOG_F(INFO, "%p Got shortinfo query for mismatching query string: %s", this,
 							query.c_str());
 					}
 				} else if (time_services_enabled_ && method == "LSL:timedata") {