From: Pierre Ossman <ossman@cendio.se>
Date: Mon, 19 Jul 2021 15:28:15 +0000 (+0200)
Subject: Avoiding final period in log messages
X-Git-Tag: v1.11.90~22
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=ab2c2ddb89af41221ba7af00823be528a9dd6624;p=tigervnc.git

Avoiding final period in log messages

Try to be consistent with existing log messages.
---

diff --git a/vncviewer/MonitorArrangement.cxx b/vncviewer/MonitorArrangement.cxx
index 04c2ab98..4c8e8db6 100644
--- a/vncviewer/MonitorArrangement.cxx
+++ b/vncviewer/MonitorArrangement.cxx
@@ -400,13 +400,13 @@ int MonitorArrangement::get_monitor_name(int m, char name[], size_t name_len)
   Fl::screen_xywh(x, y, w, h, m);
 
   if (!XQueryExtension(fl_display, "RANDR", &xi_major, &ev, &err)) {
-    vlog.info(_("Failed to get monitor name because X11 RandR could not be found."));
+    vlog.info(_("Failed to get monitor name because X11 RandR could not be found"));
     return -1;
   }
 
   XRRScreenResources *res = XRRGetScreenResources(fl_display, DefaultRootWindow(fl_display));
   if (!res) {
-    vlog.error(_("Failed to get XRRScreenResources for root window."));
+    vlog.error(_("Failed to get XRRScreenResources for root window"));
     return -1;
   }
 
@@ -427,7 +427,7 @@ int MonitorArrangement::get_monitor_name(int m, char name[], size_t name_len)
       if (monitor_found) {
         XRROutputInfo *output = XRRGetOutputInfo(fl_display, res, crtc->outputs[j]);
         if (!output) {
-          vlog.error(_("Failed to get XRROutputInfo for crtc %d, output %d."), i, j);
+          vlog.error(_("Failed to get XRROutputInfo for crtc %d, output %d"), i, j);
           continue;
         }
 
diff --git a/vncviewer/MonitorIndicesParameter.cxx b/vncviewer/MonitorIndicesParameter.cxx
index 57055f61..ebfc994a 100644
--- a/vncviewer/MonitorIndicesParameter.cxx
+++ b/vncviewer/MonitorIndicesParameter.cxx
@@ -44,7 +44,7 @@ std::set<int> MonitorIndicesParameter::getParam()
     std::vector<MonitorIndicesParameter::Monitor> monitors = fetchMonitors();
 
     if (monitors.size() <= 0) {
-        vlog.error(_("Failed to get monitors."));
+        vlog.error(_("Failed to get monitors"));
         return indices;
     }
 
@@ -75,7 +75,7 @@ bool MonitorIndicesParameter::setParam(const char* value)
         return false;
 
     if (!parseIndices(value, &indices)) {
-        vlog.error(_("Invalid FullScreenSelectedMonitors configuration."));
+        vlog.error(_("Invalid FullScreenSelectedMonitors configuration"));
         return false;
     }
 
@@ -83,7 +83,7 @@ bool MonitorIndicesParameter::setParam(const char* value)
         index = *it + 1;
 
         if (index <= 0 || index > Fl::screen_count())
-            vlog.error(_("Monitor index %d does not exist."), index);
+            vlog.error(_("Monitor index %d does not exist"), index);
     }
 
     return StringParameter::setParam(value);
@@ -97,7 +97,7 @@ bool MonitorIndicesParameter::setParam(std::set<int> indices)
     std::vector<MonitorIndicesParameter::Monitor> monitors = fetchMonitors();
 
     if (monitors.size() <=  0) {
-        vlog.error(_("Failed to get monitors."));
+        vlog.error(_("Failed to get monitors"));
         // Don't return, store the configuration anyways.
     }
 
@@ -135,12 +135,12 @@ static bool parseNumber(std::string number, std::set<int> *indices)
     int v = strtol(number.c_str(), NULL, 0);
 
     if (v <= 0) {
-        vlog.error(_("The given monitor index(%s) is too small to be valid."), number.c_str());
+        vlog.error(_("The given monitor index(%s) is too small to be valid"), number.c_str());
         return false;
     }
 
     if (v > INT_MAX) {
-        vlog.error(_("The given monitor index (%s) is too large to be valid."), number.c_str());
+        vlog.error(_("The given monitor index (%s) is too large to be valid"), number.c_str());
         return false;
     }
 
diff --git a/vncviewer/vncviewer.cxx b/vncviewer/vncviewer.cxx
index 6a84c7cb..c93a6f75 100644
--- a/vncviewer/vncviewer.cxx
+++ b/vncviewer/vncviewer.cxx
@@ -439,7 +439,7 @@ static void
 migrateDeprecatedOptions()
 {
   if (fullScreenAllMonitors) {
-    vlog.info(_("FullScreenAllMonitors is deprecated, set FullScreenMode to 'all' instead."));
+    vlog.info(_("FullScreenAllMonitors is deprecated, set FullScreenMode to 'all' instead"));
 
     fullScreenMode.setParam("all");
   }