D14288: Initial version of Clazy analyzer plugin

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Mon Jul 23 10:21:04 BST 2018


kossebau added a comment.


  Looks like good work, will have a closer look at end of week.
  
  Quick initial feedback from a first glance at the code:
  
  - misses Messages.sh file for extraction translations
  - please separate the KF includes into own group and use the camelcase forward versions (KLocalizedString, KShell, etc)
  - consider using "= default" for constructors/deconstructors

REPOSITORY
  R32 KDevelop

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

To: antonanikin, #kdevelop
Cc: kossebau, kdevelop-devel, antismap, iodelay, vbspam, njensen, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20180723/14e155d1/attachment-0001.html>


More information about the KDevelop-devel mailing list