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

Milian Wolff noreply at phabricator.kde.org
Thu Feb 2 22:19:14 UTC 2017


mwolff accepted this revision.
mwolff added a comment.
This revision is now accepted and ready to land.


  the change to kdevlcangsupport.json looks unrelated, otherwise lgtm

INLINE COMMENTS

> kdevclangsupport.json:57
>      ], 
> -    "X-KDevelop-Language": "C", 
> +    "X-KDevelop-Language": "C++", 
>      "X-KDevelop-LoadMode": "AlwaysOn", 

uhm this looks wrong - it should be both, C and C++ - did you push the change in a separate commit already?

REPOSITORY
  R32 KDevelop

BRANCH
  addclangclasshelper

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

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

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


More information about the KDevelop-devel mailing list