Changeset 14178b

Timestamp:

Oct 10, 2008, 1:36:06 PM (17 years ago)

Author:

Frederik Heber <heber@…>

Children:

3b470f

Parents:

d01f4d

Message:

BUGFIX: AssignKeySetsToFragment() had FragmentCounter-- at wrong place

+ MoleculeLeafClass::AssignKeySetsToFragment(): FragmentCounter was decreased even when no keyset had been scanned, setting it to -1, bad for the subsequent free. It has to be decreased again right after the recursive call. Begin and end messages added. Free checks for FreeList=true and ListOfLocalAtoms != NULL (safety measure).
+ MoleculeLeafClass::FillBondStructureFromReference(): No BUG here (no keyset check to prevent increase for subsequent decrease), but same changes

File:

• molecuilder/src/moleculelist.cpp (modified) (6 diffs)

molecuilder/src/moleculelist.cpp

@@ -578 +578 @@
   int AtomNo;
 
+  *out << Verbose(1) << "Begin of FillBondStructureFromReference." << endl;
   // fill ListOfLocalAtoms if NULL was given
   if (!FillListOfLocalAtoms(out, ListOfLocalAtoms, FragmentCounter, reference->AtomCount, FreeList)) {
@@ -606 +607 @@
     if (next != NULL)
       status = next->FillBondStructureFromReference(out, reference, FragmentCounter, ListOfLocalAtoms);
-  }
-  
-  FragmentCounter--;
-  if (FreeList) {
+    FragmentCounter--;
+  }
+  
+  if ((FreeList) && (ListOfLocalAtoms != NULL)) {
     // free the index lookup list
     Free((void **)&ListOfLocalAtoms[FragmentCounter], "MoleculeLeafClass::FillBondStructureFromReference - **ListOfLocalAtoms[]");
@@ -616 +617 @@
   }
   FragmentCounter--;
+  *out << Verbose(1) << "End of FillBondStructureFromReference." << endl;
   return status;
 };
@@ -706 +708 @@
   int KeySetCounter = 0;
   
+  *out << Verbose(1) << "Begin of AssignKeySetsToFragment." << endl;
   // fill ListOfLocalAtoms if NULL was given
   if (!FillListOfLocalAtoms(out, ListOfLocalAtoms, FragmentCounter, reference->AtomCount, FreeList)) {
@@ -745 +748 @@
     if (next != NULL)
       next->AssignKeySetsToFragment(out, reference, KeySetList, ListOfLocalAtoms, FragmentList, FragmentCounter, FreeList);
+    FragmentCounter--;
   } else
     *out << Verbose(1) << "KeySetList is NULL or empty." << endl;
   
-  FragmentCounter--;
-  if (FreeList) {
+  if ((FreeList) && (ListOfLocalAtoms != NULL)) {
     // free the index lookup list
     Free((void **)&ListOfLocalAtoms[FragmentCounter], "MoleculeLeafClass::AssignKeySetsToFragment - **ListOfLocalAtoms[]");
@@ -755 +758 @@
       Free((void **)&ListOfLocalAtoms, "MoleculeLeafClass::AssignKeySetsToFragment - ***ListOfLocalAtoms");
   }
+  *out << Verbose(1) << "End of AssignKeySetsToFragment." << endl;
   return status;
 };