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

diff --git a/src/FbTk/LogicCommands.hh b/src/FbTk/LogicCommands.hh
index 5e84473..c0cb938 100644
--- a/src/FbTk/LogicCommands.hh
+++ b/src/FbTk/LogicCommands.hh
@@ -38,9 +38,9 @@ public:
         m_cond(cond), m_t(t), m_f(f) { }
     void execute() {
         if (m_cond->execute()) {
-            if (*m_t) m_t->execute();
+            if (m_t) m_t->execute();
         } else
-            if (*m_f) m_f->execute();
+            if (m_f) m_f->execute();
     }
     static Command<void> *parse(const std::string &cmd, const std::string &args,
                           bool trusted);