Changeset 25aa214 for src/Filling/unittests/Ops_FillPredicateUnitTest.cpp

Timestamp:

Dec 19, 2025, 11:29:38 PM (10 days ago)

Author:

Frederik Heber <frederik.heber@…>

Branches:

Candidate_v1.7.1, stable

Children:

98ad30

Parents:

d2be22

git-author:

Frederik Heber <frederik.heber@…> (11/16/25 11:19:01)

git-committer:

Frederik Heber <frederik.heber@…> (12/19/25 23:29:38)

Message:

FIX: fragmentation hydrogens do not trigger AtomInserted.

• we had already switched off position and element changes. However, the fragmentation hydrogens could still be briefly seen on creation because the AtomInserted notification is still triggered. The World::createAtom allows now to suppress the notifcation. The switch options are put into an enum to make them very verbose in the code. This should really only be used by the HydrogenPool.

File:

• src/Filling/unittests/Ops_FillPredicateUnitTest.cpp (modified) (1 diff)

src/Filling/unittests/Ops_FillPredicateUnitTest.cpp

@@ -79 +79 @@
   // create some atoms as "neighbours"
   atoms.resize((size_t)5, NULL);
-  std::generate_n(atoms.begin(), (size_t)5, boost::bind(&World::createAtom, World::getPointer()) );
+  std::generate_n(atoms.begin(), (size_t)5, boost::bind(
+    static_cast<atom* (World::*)(World::CreateAtomNotificationType)>(&World::createAtom), 
+    World::getPointer(), 
+    World::CreateAtomNotificationType::NOTIFY_ON_CREATE_ATOM)  );
 
   // position them