From e5384b28233d2616fa2a8a34b402b828ad8a3cc8 Mon Sep 17 00:00:00 2001 From: Eike Hein Date: Sat, 25 Jun 2016 17:51:42 +0900 Subject: [PATCH] Improve comment. --- libtaskmanager/xwindowtasksmodel.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libtaskmanager/xwindowtasksmodel.cpp b/libtaskmanager/xwindowtasksmodel.cpp index 2ffcde6c6..c1e949532 100644 --- a/libtaskmanager/xwindowtasksmodel.cpp +++ b/libtaskmanager/xwindowtasksmodel.cpp @@ -471,10 +471,10 @@ QUrl XWindowTasksModel::Private::windowUrl(WId window) if (!(classClass.isEmpty() && className.isEmpty())) { int pid = NETWinInfo(QX11Info::connection(), window, QX11Info::appRootWindow(), NET::WMPid, 0).pid(); - // For KCModules, if we matched on window class, etc, we would end up matching to kcmshell4 - but we are more than likely - // interested in the actual control module. Therefore we obtain this via the commandline. This commandline may contain - // "kdeinit4:" or "[kdeinit]", so we remove these first. - // FIXME This looks like ancient old crap we can do better now. + // For KCModules, if we matched on window class, etc, we would end up matching + // to kcmshell5 itself - but we are more than likely interested in the actual + // control module. Therefore we obtain this via the commandline. This commandline + // may contain "kdeinit4:" or "[kdeinit]", so we remove these first. if (classClass == "kcmshell5") { url = serviceUrl(pid, QStringLiteral("KCModule"), QStringList() << QStringLiteral("kdeinit5:") << QStringLiteral("[kdeinit]"));