Changeset 51d33c5 for molecuilder

Timestamp:

Oct 14, 2008, 1:26:02 PM (17 years ago)

Author:

Frederik Heber <heber@…>

Children:

7a3724

Parents:

26f52a

Message:

BUGFIX: Tesselation::AddPoint() always increased Tesselation::PointsOnBoundaryCount, even if point was present already

+ we look at PointTestPair->second to determine whether point was present before or not, and only increase Tesselation::PointsOnBoundaryCount if true

File:

• molecuilder/src/boundary.cpp (modified) (1 diff)

molecuilder/src/boundary.cpp

@@ -934 +934 @@
 void Tesselation::AddPoint(atom *Walker)
 {
+  PointTestPair InsertUnique;
   BPS[0] = new class BoundaryPointSet(Walker);
-  PointsOnBoundary.insert( PointPair(Walker->nr, BPS[0]) );
-  PointsOnBoundaryCount++;
-};
+  InsertUnique = PointsOnBoundary.insert( PointPair(Walker->nr, BPS[0]) );
+  if (InsertUnique.second)  // if new point was not present before, increase counter
+    PointsOnBoundaryCount++;
+};