kossebau added inline comments.

INLINE COMMENTS

> kfontchooser.h:90
> +        ShowDifferences = 4,    ///< Display the font differences interfaces
> +        NoFixedCheckBox = 8     ///< Do not Show a checkbox to toggle 
> showing only fixed fonts
>      };

I would add a final ",", so you do not need to touch the line if any further 
flags are added in the future. Improves git blame display per line :)

REPOSITORY
  R236 KWidgetsAddons

BRANCH
  l-kfontchooser-onlyfixed-display-flag (branched from master)

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

To: ahmadsamir, #frameworks, dfaure, cfeck, bport
Cc: kossebau, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

Reply via email to