[2209] Comment about assert

98947382 · Michal 'vorner' Vaner · b2964261 · 98947382
Commit 98947382 authored Oct 29, 2012 by Michal 'vorner' Vaner
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 0 deletions

src/lib/datasrc/memory/zone_table_segment_local.cc src/lib/datasrc/memory/zone_table_segment_local.cc +5 -0

No files found.
--- a/src/lib/datasrc/memory/zone_table_segment_local.cc
+++ b/src/lib/datasrc/memory/zone_table_segment_local.cc
@@ -29,6 +29,11 @@ ZoneTableSegmentLocal::ZoneTableSegmentLocal(const RRClass& rrclass) :
 }

 ZoneTableSegmentLocal::~ZoneTableSegmentLocal() {
+    // Explicitly clear the contained data, and check memory
+    // leak.  assert() (with abort on failure) may be too harsh, but
+    // it's probably better to find more leaks initially.  Once it's stabilized
+    // we should probably revisit it.
+
    ZoneTable::destroy(mem_sgmt_, header_.getTable());
    assert(mem_sgmt_.allMemoryDeallocated());
 }