const correctness in Subscription class

diff --git a/src/CommCalls.h b/src/CommCalls.h
index 1933e0c11b6a1045a8a21ead57ab5a2d27d6403c..00785e7254ff3962379034513c4449e1132177e5 100644 (file)
--- a/src/CommCalls.h
+++ b/src/CommCalls.h
@@ -261,9 +261,6 @@ public:
     inline CommCbFunPtrCallT(int debugSection, int debugLevel,
                              const char *callName, const Dialer &aDialer);
 
-// XXX: obsolete comment?
-    // parameter cannot be const because getDialer() cannot be const
-    // getDialer() cannot because Comm IO syncWithComm() alters the object params data
     inline CommCbFunPtrCallT(const Pointer &p) :
             AsyncCall(p->debugSection, p->debugLevel, p->name),
             dialer(p->dialer)

diff --git a/src/base/Subscription.h b/src/base/Subscription.h
index f8ab6a951ce16d2d8a0d1f9fa776c1bc847f2f8c..b9deea45b90018315854e4acb2fdec3a17bffdc7 100644 (file)
--- a/src/base/Subscription.h
+++ b/src/base/Subscription.h
@@ -34,12 +34,6 @@ class CallSubscription: public Subscription
 {
 public:
     CallSubscription(const RefCount<Call_> &aCall) : call(aCall) {};
-
-// XXX: obsolete comment?
-    // cant be const sometimes because CommCbFunPtrCallT cant provide a const overload.
-    // CommCbFunPtrCallT lists why. boils down to Comm IO syncWithComm() existence
-    // NP: we still treat it as const though.
-    CallSubscription(RefCount<Call_> &aCall) : call(aCall) {};
     virtual AsyncCall::Pointer callback() { return new Call_(call); };
 //    virtual AsyncCall::Pointer callback() const { return new Call_(call); };