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, 1 insertions, 1 deletions

diff --git a/src/IconbarTool.hh b/src/IconbarTool.hh
index 90df5cf..754cd50 100644
--- a/src/IconbarTool.hh
+++ b/src/IconbarTool.hh
@@ -116,7 +116,7 @@ private:
     FbTk::IntResource 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
-    FbMenu m_menu;
+    FbTk::RefCount<FbMenu> m_menu;
     int m_alpha;
 };