ngraham added a comment.

  Why do we need this do-it-once helper function?

INLINE COMMENTS

> kcmcursortheme.cpp:466
> +            const CursorTheme *theme = selectedIndex().isValid() ? 
> m_proxyModel->theme(selectedIndex()) : nullptr;
> +            if(theme) {
> +                QString currentTheme = theme->name();

Coding style: space between `if` and `(`

REPOSITORY
  R119 Plasma Desktop

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

To: leinir, #plasma, ngraham, broulik
Cc: plasma-devel, #plasma, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

Reply via email to