dvratil added inline comments.

INLINE COMMENTS

> config_keys.h:23
>  
> -#include <KRunner/AbstractRunner>
> +static const char CONFIG_TRIGGERWORD[] = "triggerWord";
> +static const char CONFIG_ALIASES[] = "aliases";

You could use `static constexpr QStringView CONFIG_TRIGGERWORD = 
u"triggerWord"` to get something that's compile-time and easily used with 
QString API

REPOSITORY
  R114 Plasma Addons

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

To: alex, davidedmundson, ngraham, sitter, broulik, #plasma
Cc: dvratil, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to