Changeset e429dc for src/Observer/Observable.cpp

Timestamp:

Dec 19, 2011, 2:50:31 PM (14 years ago)

Author:

Frederik Heber <heber@…>

Children:

760f97c

Parents:

a1d283

Message:

FIX: Channels instance in static ChannelMap was not removed in Observable's dstor.

• this fix prevents memory leak from lazy programmers.

File:

• src/Observer/Observable.cpp (modified) (1 diff)

src/Observer/Observable.cpp

@@ -336 +336 @@
     callTable.erase(this);
   }
-}
+
+  // also kill instance in static Channels map if present
+  ChannelMap::iterator iter = NotificationChannels.find(this);
+  if (iter != NotificationChannels.end()) {
+    delete iter->second;
+    NotificationChannels.erase(iter);
+  }
+}