Log categorized only when logging category is enabled. Fixes #33

Author: Thaodan

CMakeLists.txt

@@ -12,6 +12,7 @@ set(KF_MAJOR_VERSION "6")
 
 set(CMAKE_MODULE_PATH ${ECM_MODULE_PATH} ${ECM_KDE_MODULE_DIR} ${CMAKE_MODULE_PATH})
 
+include(ECMQtDeclareLoggingCategory)
 include(KDEInstallDirs)
 include(KDECMakeSettings)
 include(KDECompilerSettings)

src/CMakeLists.txt

@@ -8,6 +8,14 @@ set(I3PagerPlugin_SRCS
         workspace.cpp
         workspace.h)
 
+ecm_qt_declare_logging_category(I3PagerPlugin_SRCS
+  HEADER i3pager_debug.h
+  IDENTIFIER I3PAGER
+  CATEGORY_NAME org.kde.I3Pager
+  DESCRIPTION "I3Pager Plasmoid"
+  EXPORT I3Pager
+)
+
 add_library(I3PagerPlugin SHARED ${I3PagerPlugin_SRCS})
 target_compile_options(I3PagerPlugin PRIVATE -fexceptions)
 
@@ -31,3 +39,6 @@ target_link_libraries(I3PagerPlugin
 # install plugin
 install(TARGETS  I3PagerPlugin DESTINATION ${KDE_INSTALL_QMLDIR}/org/kde/private/I3Pager)
 install(FILES qmldir DESTINATION ${KDE_INSTALL_QMLDIR}/org/kde/private/I3Pager)
+ecm_qt_install_logging_categories(EXPORT I3PAGER
+  FILE org.kde.i3pager.categories
+  DESTINATION ${KDE_INSTALL_LOGGINGCATEGORIESDIR})

src/i3listener.cpp

@@ -4,6 +4,7 @@
  */
 
 #include "i3listener.h"
+#include "i3pager_debug.h"
 #include <QDebug>
 #include <QLocalSocket>
 #include <QSocketNotifier>
@@ -28,25 +29,25 @@ void I3ListenerThread::handleI3Events() {
 
         // Workspace events handler
         conn.signal_workspace_event.connect([this](const i3ipc::workspace_event_t& ev) {
-            qDebug() << "workspace_event: " << (char)ev.type;
+            qCDebug(I3PAGER) << "workspace_event: " << (char)ev.type;
             if (ev.current) {
-                qDebug() << "\tSwitched to #" << ev.current->num << " - \"" << QString::fromStdString(ev.current->name) << '"';
+                qCDebug(I3PAGER) << "\tSwitched to #" << ev.current->num << " - \"" << QString::fromStdString(ev.current->name) << '"';
                 Q_EMIT workspacesChanged();
             }
         });
 
         // Mode events handler
         conn.signal_mode_event.connect([this](const i3ipc::mode_t& i3mode) {
             const auto mode = QString::fromStdString(i3mode.change);
-            qDebug() << "mode: " << mode;
+            qCDebug(I3PAGER) << "mode: " << mode;
             Q_EMIT modeChanged(mode);
         });
 
         while (!stopping) {
             conn.handle_event();
         }
     } catch (std::exception const& e) {
-        qWarning() << "Exception in i3listener handle events:" << e.what();
+        qCWarning(I3PAGER) << "Exception in i3listener handle events:" << e.what();
     }
 }
 

src/i3pager.cpp

@@ -6,6 +6,7 @@
 #include "i3pager.h"
 
 #include <i3ipc++/ipc.hpp>
+#include "i3pager_debug.h"
 #include "i3listener.h"
 #include <future>
 #include <QDebug>
@@ -16,7 +17,7 @@ I3Pager::I3Pager(QObject* parent)
     currentScreenPrivate = QString();
     mode = "default";
 
-    qDebug() << "Starting i3 listener";
+    qCDebug(I3PAGER) << "Starting i3 listener";
     this->i3ListenerThread = new I3ListenerThread(this);
     connect(i3ListenerThread, &I3ListenerThread::modeChanged, this, [=](const QString& mode) {
         this->mode = mode;
@@ -27,14 +28,14 @@ I3Pager::I3Pager(QObject* parent)
     });
     connect(i3ListenerThread, &I3ListenerThread::finished, i3ListenerThread, &QObject::deleteLater);
     i3ListenerThread->start();
-    qDebug() << "i3 listener started";
+    qCDebug(I3PAGER) << "i3 listener started";
 }
 
 I3Pager::~I3Pager() {
-    qDebug() << "I3Pager destructor";
+    qCDebug(I3PAGER) << "I3Pager destructor";
     this->i3ListenerThread->stop();
     this->i3ListenerThread->wait();
-    qDebug() << "I3Pager destructor done";
+    qCDebug(I3PAGER) << "I3Pager destructor done";
 }
 
 QList<QString> I3Pager::getScreenNames() {
@@ -46,11 +47,11 @@ QList<QString> I3Pager::getScreenNames() {
         for (auto& screen : screens) {
             if (screen->active) {
                 screenList.append(QString::fromStdString(screen->name));
-                qDebug() << "Screen name:" << QString::fromStdString(screen->name);
+                qCDebug(I3PAGER) << "Screen name:" << QString::fromStdString(screen->name);
             }
         }
     } catch (std::exception const& e) {
-        qWarning() << "Exception while retrieving screen names: " << e.what();
+        qCWarning(I3PAGER) << "Exception while retrieving screen names: " << e.what();
     }
 
     return screenList;
@@ -61,7 +62,7 @@ QList<Workspace> I3Pager::getWorkspaces(bool filterByCurrentScreen, QString orde
     try {
         i3ipc::connection conn;
         auto i3workspaceList = conn.get_workspaces();
-        qDebug() << "Loading workspaces:";
+        qCDebug(I3PAGER) << "Loading workspaces:";
 
         for (auto& i3workspace : i3workspaceList) {
             Workspace workspace;
@@ -77,8 +78,8 @@ QList<Workspace> I3Pager::getWorkspaces(bool filterByCurrentScreen, QString orde
             workspace.visible = i3workspace->visible;
             workspace.urgent = i3workspace->urgent;
 
-            qDebug() << "i3Workspace name:" << QString::fromStdString(i3workspace->name);
-            qDebug() << "Workspace:"
+            qCDebug(I3PAGER) << "i3Workspace name:" << QString::fromStdString(i3workspace->name);
+            qCDebug(I3PAGER) << "Workspace:"
                     << "id:" << workspace.id
                     << "index:" << workspace.index
                     << "name:" << workspace.name
@@ -90,7 +91,7 @@ QList<Workspace> I3Pager::getWorkspaces(bool filterByCurrentScreen, QString orde
             workspaceList.append(workspace);
         }
     } catch (std::exception const& e) {
-        qWarning() << "Exception while retrieving workspaces:" << e.what();
+        qCWarning(I3PAGER) << "Exception while retrieving workspaces:" << e.what();
     }
 
     if (filterByCurrentScreen) {

src/workspace.cpp

@@ -4,18 +4,19 @@
  */
 
 #include "workspace.h"
+#include "i3pager_debug.h"
 #include <algorithm>
 #include <QDebug>
 
 QList<Workspace> Workspace::filterByCurrentScreen(QList<Workspace> inputWorkspaces, QString currentScreen) {
     QList<Workspace> outputWorkspaces;
-    qDebug() << "Filtering by current screen:" << currentScreen;
+    qCDebug(I3PAGER) << "Filtering by current screen:" << currentScreen;
     for (auto workspace : inputWorkspaces) {
         if (currentScreen == workspace.output) {
-            qDebug() << "Added workspace:" << workspace.name << "output:" << workspace.output;
+            qCDebug(I3PAGER) << "Added workspace:" << workspace.name << "output:" << workspace.output;
             outputWorkspaces.append(workspace);
         } else {
-            qDebug() << "Removed workspace:" << workspace.name << "output:" << workspace.output;
+            qCDebug(I3PAGER) << "Removed workspace:" << workspace.name << "output:" << workspace.output;
         }
     }
     return outputWorkspaces;
@@ -30,11 +31,11 @@ QList<Workspace> Workspace::orderByName(QList<Workspace> inputWorkspaces) {
 
 QList<Workspace> Workspace::orderByOutput(QList<Workspace> inputWorkspaces, QList<QString> outputList) {
     QList<Workspace> outputWorkspaces;
-    qDebug() << "Ordering workspaces by screens:" << outputList;
+    qCDebug(I3PAGER) << "Ordering workspaces by screens:" << outputList;
     for (auto output : outputList) {
         for (auto workspace : inputWorkspaces) {
             if (workspace.output == output) {
-                qDebug() << "Added workspace:" << workspace.name << "output:" << workspace.output;
+                qCDebug(I3PAGER) << "Added workspace:" << workspace.name << "output:" << workspace.output;
                 outputWorkspaces.append(workspace);
             }
         }