Make RefCount<> more sensible

the previous version of operator*() made no sense. E.g., it violated the invariant
(*ptr).foo <=> ptr->foo. The dereferencing operator now returns a reference to the pointed-to
object, rather than a pointer to it.

I also added a bool conversion operator, which can be used in testing the NULL-ness of the
pointer. Anyone wondering if that could be done in a simpler way is encouraged to read
<http://www.artima.com/cppsource/safebool.html>.

And, finally, I removed the mutable flag from the m_data member, since it does not need it.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/CommandDialog.cc b/src/CommandDialog.cc
index bef8a63..023c22f 100644
--- a/src/CommandDialog.cc
+++ b/src/CommandDialog.cc
@@ -51,7 +51,7 @@ void CommandDialog::exec(const std::string &text){
     if (cmd.get())
         cmd->execute();
     // post execute
-    if (*m_postcommand != 0)
+    if (m_postcommand != 0)
         m_postcommand->execute();
 }