Don't destroy tracing entries. This is not necessary and even harmful, if the

code is no longer loaded, e.g. when the module has been unloaded or the tracing
buffer was reattached from a previous session.


git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@33389 a95241bf-73f2-0310-859d-f6bbb57e9c96

src/system/kernel/debug/tracing.cpp

@@ -253,8 +253,9 @@ TracingMetaData::_FreeFirstEntry()
 	if (fFirstEntry->flags & BUFFER_ENTRY) {
 		// a buffer entry -- just skip it
 	} else if (fFirstEntry->flags & ENTRY_INITIALIZED) {
-		// fully initialized TraceEntry -- destroy it
-		TraceEntry::FromTraceEntry(fFirstEntry)->~TraceEntry();
+		// Fully initialized TraceEntry: We could destroy it, but don't do so
+		// for sake of robustness. The destructors of tracing entry classes
+		// should be empty anyway.
 		fEntries--;
 	} else {
 		// Not fully initialized TraceEntry. We can't free it, since