From 61b2adba70b0a7711941c48437e907cb6e9ef2f4 Mon Sep 17 00:00:00 2001 From: Manuel Schneider Date: Thu, 16 Nov 2017 10:28:47 +0100 Subject: [PATCH] Make debug output a cmdline option --- CMakeLists.txt | 1 - lib/albertcore/src/core/albert.cpp | 27 +++++++++++---------------- 2 files changed, 11 insertions(+), 17 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 565414550..76a31d3fd 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -18,7 +18,6 @@ set(CMAKE_CXX_EXTENSIONS OFF) # Set additional compiler flags set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra -pedantic") #-Wconversion -Wunused set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} -DQT_QML_DEBUG") -set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} -DQT_NO_DEBUG_OUTPUT") include(GNUInstallDirs) diff --git a/lib/albertcore/src/core/albert.cpp b/lib/albertcore/src/core/albert.cpp index e0fedb0af..0c4959313 100644 --- a/lib/albertcore/src/core/albert.cpp +++ b/lib/albertcore/src/core/albert.cpp @@ -55,6 +55,7 @@ static SettingsWidget *settingsWidget; static TrayIcon *trayIcon; static QMenu *trayIconMenu; static QLocalServer *localServer; +static bool printDebugOutput; int Core::AlbertApp::run(int argc, char **argv) { @@ -90,9 +91,13 @@ int Core::AlbertApp::run(int argc, char **argv) { parser.addVersionOption(); parser.addOption(QCommandLineOption({"k", "hotkey"}, "Overwrite the hotkey to use.", "hotkey")); parser.addOption(QCommandLineOption({"p", "plugin-dirs"}, "Set the plugin dirs to use. Comma separated.", "directory")); + parser.addOption(QCommandLineOption({"d", "debug"}, "Print debug output.")); parser.addPositionalArgument("command", "Command to send to a running instance, if any. (show, hide, toggle)", "[command]"); parser.process(*app); + if (parser.isSet("debug")) + printDebugOutput = true; + /* * IPC/SINGLETON MECHANISM @@ -482,22 +487,13 @@ int Core::AlbertApp::run(int argc, char **argv) { /** ***************************************************************************/ void myMessageOutput(QtMsgType type, const QMessageLogContext &context, const QString &message) { switch (type) { -#ifdef QT_NO_DEBUG - case QtDebugMsg: - break; - case QtInfoMsg: - fprintf(stdout, "%s \x1b[0m[INFO]\x1b[0m %s\n", - QTime::currentTime().toString().toLocal8Bit().constData(), - message.toLocal8Bit().constData()); - fflush(stdout); - break; -#else case QtDebugMsg: - fprintf(stdout, "%s [DEBG] \x1b[3m%s -- [%s]\x1b[0m\n", - QTime::currentTime().toString().toLocal8Bit().constData(), - message.toLocal8Bit().constData(), - context.function); - fflush(stdout); + if (printDebugOutput) { + fprintf(stdout, "%s [DEBG] \x1b[3m%s\x1b[0m\n", + QTime::currentTime().toString().toLocal8Bit().constData(), + message.toLocal8Bit().constData()); + fflush(stdout); + } break; case QtInfoMsg: fprintf(stdout, "%s \x1b[32m[INFO]\x1b[0;1m %s\x1b[0m\n", @@ -505,7 +501,6 @@ void myMessageOutput(QtMsgType type, const QMessageLogContext &context, const QS message.toLocal8Bit().constData()); fflush(stdout); break; -#endif case QtWarningMsg: fprintf(stderr, "%s \x1b[33m[WARN]\x1b[0;1m %s\x1b[0m\n", QTime::currentTime().toString().toLocal8Bit().constData(),