Browse Source

Merge pull request #3650 from yay/destroy-multi-logger-no-pointer

Take logger itself, not a pointer to logger in multi-logger destructor.
gingerBill 1 year ago
parent
commit
ba8672ad29
1 changed files with 2 additions and 3 deletions
  1. 2 3
      core/log/multi_logger.odin

+ 2 - 3
core/log/multi_logger.odin

@@ -12,11 +12,10 @@ create_multi_logger :: proc(logs: ..Logger) -> Logger {
 	return Logger{multi_logger_proc, data, Level.Debug, nil}
 }
 
-destroy_multi_logger :: proc(log : ^Logger) {
+destroy_multi_logger :: proc(log: Logger) {
 	data := (^Multi_Logger_Data)(log.data)
 	delete(data.loggers)
-	free(log.data)
-	log^ = nil_logger()
+	free(data)
 }
 
 multi_logger_proc :: proc(logger_data: rawptr, level: Level, text: string,