D14288: Initial version of Clazy analyzer plugin
Friedrich W. H. Kossebau
noreply at phabricator.kde.org
Mon Jul 23 11:41:40 BST 2018
kossebau added a comment.
Oh, and then another code-style request, while at it :)
Could you make any members private, unless they really need to be accessable by subclasses?
And also split member variables off into an own section with own "private:" at the end of the class declaration, separate from any method declarations?
That should follow existing KDevelop conventions as well.
Motivation would be (besides consistency):
- by default only expose what really needs to be exposed
- keeping member variables grouped helps to more quickly understand the class design
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/c2291105/attachment.html>
More information about the KDevelop-devel
mailing list