Changeset 9f73db for molecuilder/src

Timestamp:

Oct 13, 2009, 4:42:30 PM (16 years ago)

Author:

Frederik Heber <heber@…>

Children:

57cfb7

Parents:

082da1

git-author:

Frederik Heber <heber@…> (10/13/09 16:40:34)

git-committer:

Frederik Heber <heber@…> (10/13/09 16:42:30)

Message:

Fixed broken test "Fragmentation - Fragmentation".

• BUGFIX: Code was split into CombineAllOrderListIntoOne() and FreeAllOrdersList(). In the first the combining is done and because RootStack is needed, we push_back() all pop_front() all RootKeyNr. However, the loop test was still .empty(), which never happened and an error was thrown as same keysets were tried to be inserted (assumed). Now, we store the last RootKeyNr by back() and compare the current to it. Test runs through fine.

File:

• molecuilder/src/molecule_fragmentation.cpp (modified) (5 diffs)

molecuilder/src/molecule_fragmentation.cpp

@@ -1205 +1205 @@
 
   // prepare root level (SP = 0) and a loop bond denoting Root
-  for (int i=1;i<Order;i++)
+  for (int i=Order;i--;)
     FragmentSearch.BondsPerSPCount[i] = 0;
   FragmentSearch.BondsPerSPCount[0] = 1;
@@ -1347 +1347 @@
     while (Binder->next != FragmentSearch.BondsPerSPList[2*i+1]) {
       Binder = Binder->next;
-      SP ++;
+      SP++;
     }
   }
@@ -1454 +1454 @@
   int RootNr = 0;
   int RootKeyNr = 0;
+  int StartNr = 0;
   int counter = 0;
   int NumLevels = 0;
@@ -1465 +1466 @@
     counter = FragmentList->size();
   }
-  while (!RootStack.empty()) {
+
+  StartNr = RootStack.back();
+  do {
     RootKeyNr = RootStack.front();
     RootStack.pop_front();
@@ -1477 +1480 @@
     RootStack.push_back(Walker->nr);
     RootNr++;
-  }
+  } while (RootKeyNr != StartNr);
   return counter;
 };