Trigger ClockTool every second, independent of the format string

It's hard to hit right moment to update the clock correctly: Either we are a
bit too early and the 'old time' is the same as the 'new time' and the clock
looks 'a bit off' or we are too late and the clock is 'a bit off'. This is
especially the case for format strings which do not show the second and thus
are updated only at the full minute (see bug #1082): if fluxbox does not
update the clock 'now' it would do it one minute later and thus the clock
might show the wrong time for roughly 1 minute.

Instead of coming up with something immensely clever we just trigger the
ClockTool every second. If no update of the shown time is needed fluxbox won't
do anything.

Some minor code reordering as byproduct.

1 files changed, 8 insertions, 10 deletions

diff --git a/src/ClockTool.hh b/src/ClockTool.hh
index 4f89b36..2872f33 100644
--- a/src/ClockTool.hh
+++ b/src/ClockTool.hh
@@ -72,18 +72,16 @@ private:
     void reRender();
     void updateSizing();
 
-    FbTk::TextButton m_button;
+    FbTk::TextButton                    m_button;
 
-    const FbTk::ThemeProxy<ToolTheme> &m_theme;
-    BScreen &m_screen;
-    Pixmap m_pixmap;
-    FbTk::Timer m_timer;
+    const FbTk::ThemeProxy<ToolTheme>&  m_theme;
+    BScreen&                            m_screen;
+    Pixmap                              m_pixmap;
+    FbTk::Timer                         m_timer;
 
-    FbTk::Resource<std::string> m_timeformat;
-
-    FbTk::StringConvertor m_stringconvertor;
-
-    FbTk::SignalTracker m_tracker;
+    FbTk::Resource<std::string>         m_timeformat;
+    FbTk::StringConvertor               m_stringconvertor;
+    FbTk::SignalTracker                 m_tracker;
 };
 
 #endif // CLOCKTOOL_HH