Changeset c8cb0d for ThirdParty/CodePatterns/src/Observer/unittests/stubs/ObserverStub.cpp

Timestamp:

Dec 26, 2025, 9:40:14 PM (2 days ago)

Author:

Frederik Heber <frederik.heber@…>

Branches:

Candidate_v1.7.1, stable

Children:

033646

Parents:

f2d5ce

git-author:

Frederik Heber <frederik.heber@…> (12/10/25 16:15:28)

git-committer:

Frederik Heber <frederik.heber@…> (12/26/25 21:40:14)

Message:

Streamlines channel creation in Observables.

• CodePatterns is now version 1.3.4.
• we no longer need to add the channels manually in the cstor of a class that derives from Observable. Instead, we just need to pass the maximum number of channels (as they are typically enumerated anyway) and they are generated and added.
• added mutex protection when inserting.
• adjusted class Relay to forward similar convenience cstors.
• adjusted all call sites in molecuilder.

File:

• ThirdParty/CodePatterns/src/Observer/unittests/stubs/ObserverStub.cpp (modified) (2 diffs)

ThirdParty/CodePatterns/src/Observer/unittests/stubs/ObserverStub.cpp

@@ -133 +133 @@
 
 NotificationObservable::NotificationObservable() :
-  Observable("NotificationObservable")
-{
-  Channels *OurChannel = new Channels();
-  Observable::insertNotificationChannel( std::make_pair(this, OurChannel) );
-  OurChannel->addChannel(Operation1Notify);
-  OurChannel->addChannel(Operation2Notify);
-}
+  Observable("NotificationObservable", { Operation1Notify, Operation2Notify })
+{}
 
 NotificationObservable::~NotificationObservable()
@@ -242 +237 @@
 
 RelayNotification::RelayNotification() :
-    Relay(std::string("RelayTest"))
-{
-  Channels *OurChannel = new Channels();
-  Observable::insertNotificationChannel( std::make_pair(this, OurChannel) );
-  OurChannel->addChannel(NotificationObservable::Operation1Notify);
-  OurChannel->addChannel(NotificationObservable::Operation2Notify);
-}
+    Relay(std::string("RelayTest"), { NotificationObservable::Operation1Notify, NotificationObservable::Operation2Notify })
+{}
 
 RelayNotification::~RelayNotification()