Store menus if smart pointers (RefCount)

This was originally intended to be a bugfix for an memory error reported by valgrind (accessing
freed memory). While debugging it, I found the menu ownership semantics confusing
(setInternalMenu() et al.), so I decided to get rid of it and store it in smart pointers
everywhere.
Looking back, I'm not sure if this was worth all the trouble, but the good news is that the
valgrind error disappeared. :)

1 files changed, 3 insertions, 3 deletions

diff --git a/src/Workspace.hh b/src/Workspace.hh
index 2dabb3e..829bdf0 100644
--- a/src/Workspace.hh
+++ b/src/Workspace.hh
@@ -65,8 +65,8 @@ public:
     BScreen &screen() { return m_screen; }
     const BScreen &screen() const { return m_screen; }
 
-    FbTk::Menu &menu() { return m_clientmenu; }
-    const FbTk::Menu &menu() const { return m_clientmenu; }
+    FbTk::RefCount<FbTk::Menu> menu() { return m_clientmenu; }
+    FbTk::RefCount<const FbTk::Menu> menu() const { return m_clientmenu; }
     ///    name of this workspace
     const FbTk::FbString &name() const { return m_name; }
     /**
@@ -86,7 +86,7 @@ private:
 
     Windows m_windowlist;
     FbTk::Signal<> m_clientlist_sig;
-    ClientMenu m_clientmenu;
+    FbTk::RefCount<ClientMenu> m_clientmenu;
 
     FbTk::FbString m_name;  ///< name of this workspace
     unsigned int m_id;    ///< id, obsolete, this should be in BScreen