[Differential] [Requested Changes To] D4039: Add 'override' keyword only for new standards (BUG 372280)

[Kevin Funk]

kfunk requested changes to this revision.
kfunk added a comment.
This revision now requires changes to proceed.


  Good idea in general. This looks a bit complex for what you try to achieve though.
  
  But I think it would be easier if you'd go through `KDevelop::IDefinesAndIncludesManager::parserArguments(QString)` for figuring  out the cpp flags, no? Did you try that?

REPOSITORY
  R32 KDevelop

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

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

To: denyssenkin, #kdevelop, kfunk
Cc: kdevelop-devel, Pilzschaf, akshaydeo, surgenight, arrowdodger
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170109/df8509af/attachment-0001.html>