[Differential] [Commented On] D4359: Add ClangClassHelper, to restore features of CppClassHelper

[Friedrich W. H. Kossebau]

kossebau added inline comments.

INLINE COMMENTS

> kossebau wrote in clangclasshelper.cpp:137
> Well, it means when reading the code one has to jump from the place where it is called to the place where the template is called.
> I see the value of DRY, just that I now and then trade repeating for ease of reading flow :)
> 
> Picked up your proposal, works fine. Not sure about that naming of the template method, as it overloads a member method, but the namespace prefix `::` should make it obvious also to the human reader what is called, no?

> where it is called to the place where the template is called.

where it is called to the place where the template is declared.

REPOSITORY
  R32 KDevelop

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: kossebau, #kdevelop, mwolff
Cc: mwolff, arrowdodger, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170202/b832adde/attachment.html>