Changeset 746bb4 for molecuilder/src/periodentafel.cpp

Timestamp:

Oct 15, 2009, 8:10:10 AM (16 years ago)

Author:

Frederik Heber <heber@…>

Children:

543794

Parents:

13ecff

Message:

Fixed two memory leaks.

• FIX: if the element database could not be parsed, in LoadPeriodentafel() still one element would be allocated, Loading failure admonished and then not free'd. Fixed
• FIX: the config was allocated in main not by a new, but fixed. In its constructor Malloc calls were present. Hence, the memory was not free'd. As fixed types are free'd first at the very end of the code. Hence before any MemoryUsageObserver::getUsedMemorySize() calls ... that's why 1614 bytes were always claimed as still allocated. Fixed.
• valgrind does not admonish any leaks (however, some errors) anymore when molecuilder is started and immediately quitted.

Signed-off-by: Frederik Heber <heber@…>

File:

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

molecuilder/src/periodentafel.cpp

@@ -215 +215 @@
         cout << "Could not parse element: ";
         neues->Output((ofstream *)&cout);
+        delete(neues);
       }
     }