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