Attach a modifiedSig handler to the session.keyFile resource

5 files changed, 12 insertions, 21 deletions

diff --git a/src/FbCommands.cc b/src/FbCommands.cc
index c789b6b..119f378 100644
--- a/src/FbCommands.cc
+++ b/src/FbCommands.cc
@@ -460,7 +460,7 @@ BindKeyCmd::BindKeyCmd(const string &keybind):m_keybind(keybind) { }
 void BindKeyCmd::execute() {
     Keys* keys = Fluxbox::instance()->keys();
     if (keys && keys->addBinding(m_keybind)) {
-        ofstream ofile(keys->filename().c_str(), ios::app);
+        ofstream ofile(Fluxbox::instance()->getKeysResource()->c_str(), ios::app);
         if (!ofile)
             return;
         ofile<<m_keybind<<endl;
diff --git a/src/Keys.cc b/src/Keys.cc
index c4ad5ea..717a8fc 100644
--- a/src/Keys.cc
+++ b/src/Keys.cc
@@ -188,7 +188,11 @@ Keys::Keys():
     m_reloader(new FbTk::AutoReloadHelper()),
     m_keylist(0),
     next_key(0), saved_keymode(0) {
+
     m_reloader->setReloadCmd(FbTk::RefCount<FbTk::Command<void> >(new FbTk::SimpleCommand<Keys>(*this, &Keys::reload)));
+    m_reloader->setMainFile(*Fluxbox::instance()->getKeysResource());
+    join(Fluxbox::instance()->getKeysResource().modifiedSig(),
+            MemFun(*m_reloader, &FbTk::AutoReloadHelper::setMainFile));
 }
 
 Keys::~Keys() {
@@ -282,8 +286,9 @@ void Keys::grabWindow(Window win) {
 void Keys::reload() {
     // an intentionally empty file will still have one root mapping
     bool firstload = m_map.empty();
+    const std::string filename = FbTk::StringUtil::expandFilename(*Fluxbox::instance()->getKeysResource());
 
-    if (m_filename.empty()) {
+    if (filename.empty()) {
         if (firstload)
             loadDefaults();
         return;
@@ -291,12 +296,12 @@ void Keys::reload() {
 
     FbTk::App::instance()->sync(false);
 
-    if (! FbTk::FileUtil::isRegularFile(m_filename.c_str())) {
+    if (! FbTk::FileUtil::isRegularFile(filename.c_str())) {
         return;
     }
 
     // open the file
-    ifstream infile(m_filename.c_str());
+    ifstream infile(filename.c_str());
     if (!infile) {
         if (firstload)
             loadDefaults();
@@ -609,16 +614,6 @@ void Keys::unregisterWindow(Window win) {
     m_window_map.erase(win);
 }
 
-/**
- deletes the tree and load configuration
- returns true on success else false
-*/
-void Keys::reconfigure() {
-    m_filename = FbTk::StringUtil::expandFilename(Fluxbox::instance()->getKeysFilename());
-    m_reloader->setMainFile(m_filename);
-    m_reloader->checkReload();
-}
-
 void Keys::regrab() {
     setKeyMode(m_keylist);
 }
diff --git a/src/Keys.hh b/src/Keys.hh
index 7a6b3b8..076387b 100644
--- a/src/Keys.hh
+++ b/src/Keys.hh
@@ -24,6 +24,7 @@
 
 #include "FbTk/NotCopyable.hh"
 #include "FbTk/RefCount.hh"
+#include "FbTk/Signal.hh"
 
 #include <X11/Xlib.h>
 #include <string>
@@ -36,7 +37,7 @@ namespace FbTk {
     class AutoReloadHelper;
 }
 
-class Keys:private FbTk::NotCopyable  {
+class Keys: private FbTk::NotCopyable, private FbTk::SignalTracker  {
 public:
 
     // contexts for events
@@ -80,7 +81,6 @@ public:
     /// grab keys again when keymap changes
     void regrab();
 
-    const std::string& filename() const { return m_filename; }
     /**
        Load configuration from file
     */
@@ -88,7 +88,6 @@ public:
     /**
        Reload configuration if keys file has changed
     */
-    void reconfigure();
     void keyMode(const std::string& keyMode);
 
     bool inKeychain() const { return saved_keymode != 0; }
@@ -114,7 +113,6 @@ private:
 
 
     // member variables
-    std::string m_filename;
     FbTk::AutoReloadHelper* m_reloader;
     RefKey m_keylist;
     keyspace_t m_map;
diff --git a/src/fluxbox.cc b/src/fluxbox.cc
index 7b46123..1dfcab8 100644
--- a/src/fluxbox.cc
+++ b/src/fluxbox.cc
@@ -358,7 +358,6 @@ Fluxbox::Fluxbox(int argc, char **argv,
     // Create keybindings handler and load keys file
     // Note: this needs to be done before creating screens
     m_key.reset(new Keys);
-    m_key->reconfigure();
 
     vector<int> screens;
     int i;
@@ -1202,7 +1201,6 @@ void Fluxbox::reconfigure() {
 
 void Fluxbox::real_reconfigure() {
     STLUtil::forAll(m_screen_list, mem_fun(&BScreen::reconfigure));
-    m_key->reconfigure();
 }
 
 BScreen *Fluxbox::findScreen(int id) {
diff --git a/src/fluxbox.hh b/src/fluxbox.hh
index b028363..8ce82de 100644
--- a/src/fluxbox.hh
+++ b/src/fluxbox.hh
@@ -121,8 +121,8 @@ public:
     const std::string &getStyleOverlayFilename() const { return *m_rc_styleoverlayfile; }
 
     const std::string &getMenuFilename() const { return *m_rc_menufile; }
-    const std::string &getKeysFilename() const { return *m_rc_keyfile; }
     FbTk::StringResource &getAppsResource() { return m_rc_appsfile; }
+    FbTk::StringResource &getKeysResource() { return m_rc_keyfile; }
     int colorsPerChannel() const { return *m_rc_colors_per_channel; }
     int getTabsPadding() const { return *m_rc_tabs_padding; }