Fix for a merged pull request: close log file only if it's been opened in Logger destructor

ivansafrin · ivansafrin · commit 53466270d85c · 2014-12-28T11:29:19.000-05:00
diff --git a/Core/Contents/Source/PolyLogger.cpp b/Core/Contents/Source/PolyLogger.cpp
@@ -46,11 +46,13 @@ LoggerEvent::~LoggerEvent() {
 
 Logger::Logger() : EventDispatcher() {
 	logToFile = false;
-	//logFile = fopen("poly.log", "w");
+    logFile = NULL;
 }
 
 Logger::~Logger() {
-	fclose(logFile);
+    if(logFile) {
+        fclose(logFile);
+    }
 }
 
 void Logger::logBroadcast(String message) {

Original file line number	Diff line number	Diff line change
`@@ -46,11 +46,13 @@ LoggerEvent::~LoggerEvent() {`
`46`	`46`
`47`	`47`	`Logger::Logger() : EventDispatcher() {`
`48`	`48`	`logToFile = false;`
`49`		`- //logFile = fopen("poly.log", "w");`
	`49`	`+ logFile = NULL;`
`50`	`50`	`}`
`51`	`51`
`52`	`52`	`Logger::~Logger() {`
`53`		`- fclose(logFile);`
	`53`	`+ if(logFile) {`
	`54`	`+ fclose(logFile);`
	`55`	`+ }`
`54`	`56`	`}`
`55`	`57`
`56`	`58`	`void Logger::logBroadcast(String message) {`