smw/deskflow
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: Use QTextStream to write to stdout and stderr

Browse source 
Only use `Qt::endl` when writing to the output stream; `Q_EMIT newLine` should not include a trailing newline because `appendPlainText` doesn't expect one.
This commit is contained in:
Nick Bolton 2025-04-14 10:13:16 +01:00 committed by Chris Rizzitello
parent c8408c4823
commit c62b4ec068
1 changed files with 8 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
src/lib/gui/Logger.cpp
View file

@ -36,18 +36,21 @@ QString printLine(FILE *out, const QString &type, const QString &message, const
stream << "\n\t" + fileLine;
}
auto logLineBytes = logLine.toUtf8();
auto logLine_c = logLineBytes.constData();
// We must return a non-terminated log line, but before returning,
// stdout/stderr and Windows debug output all expect a terminated line.
QString terminatedLogLine = logLine;
QTextStream terminatedStream(&terminatedLogLine);
terminatedStream << Qt::endl;
#if defined(Q_OS_WIN)
// Debug output is viewable using either VS Code, Visual Studio, DebugView, or
// DbgView++ (only one can be used at once). It's important to send output to
// the debug output API, because it's difficult to view stdout and stderr from
// a Windows GUI app.
OutputDebugStringA(logLine_c);
OutputDebugStringA(terminatedLogLine.toLocal8Bit().constData());
#else
fprintf(out, "%s", logLine_c);
fflush(out);
QTextStream outStream(out);
outStream << terminatedLogLine;
#endif
return logLine;