Changeset c8cb0d for src/LinkedCell/unittests

Timestamp:

Dec 26, 2025, 9:40:14 PM (6 weeks 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.

Location:

src/LinkedCell/unittests/stubs

Files:

• ObserverBoxStub.cpp (modified) (1 diff)
• WorldTimeStub.cpp (modified) (1 diff)

src/LinkedCell/unittests/stubs/ObserverBoxStub.cpp

@@ -44 +44 @@
 
 Box::Box(RealSpaceMatrix _M) :
-  Observable("Box"),
+  Observable("Box", NotificationType_MAX),
   M( new RealSpaceMatrix(_M) ),
   Minv( new RealSpaceMatrix )
 {
-  // observable stuff
-  Channels *OurChannel = new Channels;
-  Observable::insertNotificationChannel( std::make_pair(static_cast<Observable *>(this), OurChannel) );
-  // add instance for each notification type
-  OurChannel->addChannel(Box::MatrixChanged);
   *Minv = M->invert();
 }

src/LinkedCell/unittests/stubs/WorldTimeStub.cpp

@@ -45 +45 @@
 
 WorldTime::WorldTime() :
-  Observable("WorldTime"),
+  Observable("WorldTime", NotificationType_MAX),
   StepWidth(0.)
-{
-  // observable stuff
-  Channels *OurChannel = new Channels;
-  Observable::insertNotificationChannel( std::make_pair(static_cast<Observable *>(this), OurChannel) );
-  // add instance for each notification type
-  for (size_t type = 0; type < NotificationType_MAX; ++type)
-    OurChannel->addChannel(type);
-}
+{}
 
 WorldTime::~WorldTime()