Emit the "modified" signal when a resource value is changed through C++ code

5 files changed, 12 insertions, 8 deletions

diff --git a/src/FbTk/Resource.hh b/src/FbTk/Resource.hh
index 37a7ed3..7a797a1 100644
--- a/src/FbTk/Resource.hh
+++ b/src/FbTk/Resource.hh
@@ -224,7 +224,11 @@ public:
             setDefaultValue();
         }
     }
-    Accessor<T> &operator =(const T& newvalue) { m_value = newvalue; return *this;}
+    Accessor<T> &operator =(const T& newvalue) {
+        m_value = newvalue;
+        m_modified_sig.emit(m_value);
+        return *this;
+    }
     /// specialized, must be implemented
     /// @return string value of resource
     std::string getString() const { return Traits::toString(m_value); }
@@ -243,7 +247,7 @@ public:
 
     operator T() const { return m_value; }
     T& get() { return m_value; }
-    T& operator*() { return m_value; }
+    Accessor<T>& operator*() { return *this; }
     const T& operator*() const { return m_value; }
     T *operator->() { return &m_value; }
     const T *operator->() const { return &m_value; }
diff --git a/src/IconbarTool.cc b/src/IconbarTool.cc
index 070e45b..73f0003 100644
--- a/src/IconbarTool.cc
+++ b/src/IconbarTool.cc
@@ -242,7 +242,7 @@ IconbarTool::IconbarTool(const FbTk::FbWindow &parent, IconbarTheme &theme,
               screen.name() + ".iconbar.mode"),
     m_rc_alignment(screen.resourceManager(), FbTk::Container::RELATIVE,
                    screen.name() + ".iconbar.alignment"),
-    m_rc_client_width(screen.resourceManager(), 128, screen.name() + ".iconbar.iconWidth"),
+    m_rc_client_width(screen.resourceManager(), 128, screen.name() + ".iconbar.iconWidth", FbTk::RangedIntTraits(10, 400)),
     m_rc_client_padding(screen.resourceManager(), 10,
                    screen.name() + ".iconbar.iconTextPadding"),
     m_rc_use_pixmap(screen.resourceManager(), true, screen.name() + ".iconbar.usePixmap"),
@@ -380,7 +380,6 @@ void IconbarTool::update(UpdateReason reason, Focusable *win) {
 
     m_icon_container.setAlignment(*m_rc_alignment);
 
-    *m_rc_client_width = FbTk::Util::clamp(*m_rc_client_width, 10, 400);
     m_icon_container.setMaxSizePerClient(*m_rc_client_width);
 
     // lock graphic update
diff --git a/src/IconbarTool.hh b/src/IconbarTool.hh
index 754cd50..9091a91 100644
--- a/src/IconbarTool.hh
+++ b/src/IconbarTool.hh
@@ -113,7 +113,7 @@ private:
     FbTk::StringResource m_rc_mode;
     /// alignment of buttons
     FbTk::Resource<FbTk::Container::Alignment, FbTk::EnumTraits<FbTk::Container::Alignment> > m_rc_alignment;
-    FbTk::IntResource m_rc_client_width; ///< size of client button in LEFT/RIGHT mode
+    FbTk::RangedIntResource m_rc_client_width; ///< size of client button in LEFT/RIGHT mode
     FbTk::UIntResource m_rc_client_padding; ///< padding of the text
     FbTk::BoolResource m_rc_use_pixmap; ///< if iconbar should use win pixmap or not
     FbTk::RefCount<FbMenu> m_menu;
diff --git a/src/Screen.cc b/src/Screen.cc
index 37381cd..fa8df19 100644
--- a/src/Screen.cc
+++ b/src/Screen.cc
@@ -822,7 +822,7 @@ void BScreen::reconfigure() {
 
     // and update their names
     for(size_t i = 0; i < std::min(m_workspaces_list.size(), resource.workspace_names->size()); ++i)
-        m_workspaces_list[i]->setName( (*resource.workspace_names)[i] );
+        m_workspaces_list[i]->setName( resource.workspace_names.get()[i] );
 
     // update menu filenames
     m_rootmenu->reloadHelper()->setMainFile(fluxbox->getMenuFilename());
@@ -862,7 +862,8 @@ void BScreen::reconfigureTabs() {
 void BScreen::updateWorkspaceName(unsigned int w) {
     Workspace *space = getWorkspace(w);
     if (space) {
-        (*resource.workspace_names)[w] = space->name();
+        resource.workspace_names.get()[w] = space->name();
+        // XXX use resource.workspace_names.modifiedSig() instead
         m_workspacenames_sig.emit(*this);
         Fluxbox::instance()->save_rc();
     }
diff --git a/src/Window.cc b/src/Window.cc
index 771ac7c..0081e75 100644
--- a/src/Window.cc
+++ b/src/Window.cc
@@ -3506,7 +3506,7 @@ void FluxboxWindow::updateButtons() {
             need_update = true;
         else {
             for (size_t j=0; j < new_size && !need_update; j++) {
-                if ((*(*titlebar_side[i]))[j] != m_titlebar_buttons[i][j])
+                if ((*titlebar_side[i]).get()[j] != m_titlebar_buttons[i][j])
                     need_update = true;
             }
         }