Apply suggestions from code review

Co-authored-by: Charles-Henri Bruyand <charleshenri.bruyand+github@gmail.com>

diff --git a/pdns/arguments.cc b/pdns/arguments.cc
index b6769ed046af1da6a9b094ae181c3f9a2a0319eb..b22da985313d07216e263caa9589df7ee5f33487 100644 (file)
--- a/pdns/arguments.cc
+++ b/pdns/arguments.cc
@@ -530,7 +530,7 @@ bool ArgvMap::file(const char *fname, bool lax, bool included)
 
   if(!parseFile(fname, "", lax)) {
     SLOG(g_log << Logger::Warning << "Unable to open " << fname << std::endl,
-         d_log->error(Logr::Warning, "Unable to open", "name", Logging::Loggable(fname)));
+         d_log->error(Logr::Warning, "Unable to open file", "name", Logging::Loggable(fname)));
     return false;
   }
 
@@ -541,7 +541,7 @@ bool ArgvMap::file(const char *fname, bool lax, bool included)
     for(const std::string& fn :  extraConfigs) {
       if (!file(fn.c_str(), lax, true)) {
         SLOG(g_log << Logger::Error << fn << " could not be parsed" << std::endl,
-             d_log->info(Logr::Error, "Could not be parsed", "name", Logging::Loggable(fn)));
+             d_log->info(Logr::Error, "Unable to parse config file", "name", Logging::Loggable(fn)));
         throw ArgException(fn + " could not be parsed");
       }
     }
@@ -576,7 +576,7 @@ void ArgvMap::gatherIncludes(std::vector<std::string> &extraConfigs) {
       if (stat(name.c_str(), &st) || !S_ISREG(st.st_mode)) {
         string msg = name + " is not a regular file";
         SLOG(g_log << Logger::Error << msg << std::endl,
-             d_log->info(Logr::Error, "Is not a regular file", "name", Logging::Loggable(name)));
+             d_log->info(Logr::Error, "Unable to open non-regular file", "name", Logging::Loggable(name)));
         closedir(dir);
         throw ArgException(msg);
       }

diff --git a/pdns/recursordist/rec-main.cc b/pdns/recursordist/rec-main.cc
index f5819dffb67ec8686a46cf1cffa8a9b9099612e3..b8b6b3e65f629af17d66fefc9daa506e597427d9 100644 (file)
--- a/pdns/recursordist/rec-main.cc
+++ b/pdns/recursordist/rec-main.cc
@@ -1238,7 +1238,7 @@ static int serviceMain(int argc, char* argv[], std::shared_ptr<Logr::Logger>& lo
   }
   catch (std::exception& e) {
     SLOG(g_log << Logger::Error << "Assigning local query addresses: " << e.what(),
-         log->error(Logr::Error, e.what(), "Assigning local query address"));
+         log->error(Logr::Error, e.what(), "Unable to assign local query address"));
     exit(99);
   }