mvourlakos added inline comments.

INLINE COMMENTS

> appmenu.cpp:211
>      hideMenu(); // hide window decoration menu if exists
> -
> +return;
>      KConfigGroup 
> config(KSharedConfig::openConfig(QStringLiteral("kdeglobals")), 
> QStringLiteral("Appmenu Style"));

is this ok? isnt that return blocks any code below?

REPOSITORY
  R120 Plasma Workspace

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

To: mart, #plasma
Cc: mvourlakos, broulik, plasma-devel, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

Reply via email to