[Xfce4-commits] [panel-plugins/xfce4-hardware-monitor-plugin] 35/96: PreferencesWindow::on_selection_changed default color should be fine, all monitors should have a set colour anyway

noreply at xfce.org noreply at xfce.org
Thu Nov 27 22:20:40 CET 2014


This is an automated email from the git hooks/post-receive script.

nick pushed a commit to branch master
in repository panel-plugins/xfce4-hardware-monitor-plugin.

commit 75fbe6d7efed5884157e78d4e99811b4b4674fb7
Author: Omega Weapon <OmegaPhil at gmail.com>
Date:   Mon Nov 4 19:56:36 2013 +0000

    PreferencesWindow::on_selection_changed default color should be fine, all monitors should have a set colour anyway
---
 src/preferences-window.cpp |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/preferences-window.cpp b/src/preferences-window.cpp
index f69a528..287f452 100644
--- a/src/preferences-window.cpp
+++ b/src/preferences-window.cpp
@@ -797,7 +797,7 @@ void PreferencesWindow::on_selection_changed()
   // Making sure the selection is available
   if (sel)
   {
-    unsigned int color = 0;  // TODO: Is this an acceptable default? Can't seem to see how the normal default happens
+    unsigned int color = 0;
 
     // Loading up new monitor colour
     // Fetching assigned settings group
@@ -814,7 +814,7 @@ void PreferencesWindow::on_selection_changed()
 
       // Loading color
       xfce_rc_set_group(settings, mon_dir.c_str());
-      color = xfce_rc_read_int_entry(settings, "color", 0);  // TODO: Is this an acceptable default? Can't seem to see how the normal default happens
+      color = xfce_rc_read_int_entry(settings, "color", 0);
 
       // Close settings file
       xfce_rc_close(settings);

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.


More information about the Xfce4-commits mailing list