ngraham added inline comments.

INLINE COMMENTS

> workspaceoptions.cpp:51
>  
> -/*ConfigModule functions*/
> +// ***ConfigModule functions
>  void KCMWorkspaceOptions::load()

Why all the asterisks?

> workspaceoptions.h:58
>      // QML variables
> -    bool m_ostateToolTip;   // Original state
> -    bool m_stateToolTip;    // Current state
> +    bool m_toolTipOriginalState;   // Original state
> +    bool m_toolTipCurrentState;    // Current state

The inline comments are no longer needed now that the variable name is 
self-documenting.

REPOSITORY
  R119 Plasma Desktop

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

To: furkantokac, ngraham, romangg, #plasma
Cc: zzag, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

Reply via email to