Fix checking for force stderr logging and asume stderr has console

JohannAnhofer · andibacher · commit 82778de1bade · 2024-12-08T21:32:44.000+01:00
diff --git a/src/log4qt/consoleappender.cpp b/src/log4qt/consoleappender.cpp
@@ -132,7 +132,8 @@ void ConsoleAppender::closeStream()
 void ConsoleAppender::append(const LoggingEvent &event)
 {
 #ifdef Q_OS_WIN
-    if (!GetConsoleWindow() && !qEnvironmentVariableIsEmpty("QT_ASSUME_STDERR_HAS_CONSOLE"))
+    if (GetConsoleWindow() == nullptr &&
+        qEnvironmentVariableIntValue("QT_ASSUME_STDERR_HAS_CONSOLE") == 0)
     {
         // if console is blocked by debugger use OutputDebugString
         Q_ASSERT_X(layout(), "ConsoleAppender::append()", "Layout must not be null");

Original file line number	Diff line number	Diff line change
`@@ -132,7 +132,8 @@ void ConsoleAppender::closeStream()`
`132`	`132`	`void ConsoleAppender::append(const LoggingEvent &event)`
`133`	`133`	`{`
`134`	`134`	`#ifdef Q_OS_WIN`
`135`		`- if (!GetConsoleWindow() && !qEnvironmentVariableIsEmpty("QT_ASSUME_STDERR_HAS_CONSOLE"))`
	`135`	`+ if (GetConsoleWindow() == nullptr &&`
	`136`	`+ qEnvironmentVariableIntValue("QT_ASSUME_STDERR_HAS_CONSOLE") == 0)`
`136`	`137`	`{`
`137`	`138`	`// if console is blocked by debugger use OutputDebugString`
`138`	`139`	`Q_ASSERT_X(layout(), "ConsoleAppender::append()", "Layout must not be null");`