anthonyfieroni added a comment.

  Will be better if you use QLatin1Char('\n'), "\n" will call strlen on which 
is unneeded. Also QStringLiteral is missing on some strings.

REPOSITORY
  R238 KDocTools

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

To: mlaurent, dfaure
Cc: anthonyfieroni, kde-frameworks-devel, kde-doc-english, LeGast00n, gennad, 
fbampaloukas, GB_2, michaelh, ngraham, bruns, skadinna

Reply via email to