Changeset bc2698 for src/Patterns/unittests/stubs/ObserverStub.cpp

Timestamp:

Dec 13, 2011, 9:43:39 AM (14 years ago)

Author:

Frederik Heber <heber@…>

Children:

e2e035e

Parents:

e7352a5

git-author:

Frederik Heber <heber@…> (12/01/11 09:09:18)

git-committer:

Frederik Heber <heber@…> (12/13/11 09:43:39)

Message:

NotificationChannels is now static entity.

• this is in line with the other static entities on Observable that are intended to save memory (before we needed 8 bytes for the pointer in each and every Observable instances, whether it had channels or not.)
• Modified NOTIFY macro to accomodate for the changes.
• enhanced documentation of class Notifications.

File:

• src/Patterns/unittests/stubs/ObserverStub.cpp (modified) (1 diff)

src/Patterns/unittests/stubs/ObserverStub.cpp

@@ -132 +132 @@
   Observable("NotificationObservable")
 {
-  NotificationChannels = new Channels(this);
-  NotificationChannels->addChannel(Operation1Notify);
-  NotificationChannels->addChannel(Operation2Notify);
+  Channels *OurChannel = new Channels(this);
+  NotificationChannels.insert( std::make_pair(this, OurChannel) );
+  OurChannel->addChannel(Operation1Notify);
+  OurChannel->addChannel(Operation2Notify);
 }
 
 NotificationObservable::~NotificationObservable()
 {
-  delete NotificationChannels;
+  NotificationChannels.erase(this);
 }