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