broulik added inline comments.
Restricted Application edited projects, added KWin; removed Plasma.

INLINE COMMENTS

> dbusinterface.cpp:199
> +                const QVariantMap ret{
> +                    {QStringLiteral("resourceClass"), c->resourceClass()},
> +                    {QStringLiteral("resourceName"), c->resourceName()},

Can you use introspection to fill this `QVariantMap` rather than manually doing 
this?

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D10490

To: graesslin, #kwin, #plasma
Cc: broulik, plasma-devel, kwin, iodelay, bwowk, ZrenBot, progwolff, 
lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart

Reply via email to