https://bugs.kde.org/show_bug.cgi?id=372280

Kevin Funk <kf...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://commits.kde.org/kde |https://commits.kde.org/kde
                   |velop/0636cfdd8a93617cbcf2e |velop/27cca2f34f407d02212b1
                   |fe2c9d74eab21a0b97b         |5deb69d4506ce30d8f0

--- Comment #4 from Kevin Funk <kf...@kde.org> ---
Git commit 27cca2f34f407d02212b15deb69d4506ce30d8f0 by Kevin Funk, on behalf of
Amish Naidu.
Committed on 03/12/2018 at 10:57.
Pushed by kfunk into branch '5.3'.

Don't add 'override' specifier for non-modern project settings

Summary:
Override code completion will only add the override specifier if the parser
settings don't have either -std=c++03 or 98.

Supersedes: D4039

Test Plan: Run `test_codecompletion testOverrideExecute`

Reviewers: kfunk

Reviewed By: kfunk

Subscribers: kfunk, kdevelop-devel

Tags: #kdevelop

Differential Revision: https://phabricator.kde.org/D16773

(cherry picked from commit 0636cfdd8a93617cbcf2efe2c9d74eab21a0b97b)

M  +27   -1    plugins/clang/codecompletion/context.cpp
M  +65   -0    plugins/clang/tests/test_codecompletion.cpp
M  +2    -0    plugins/clang/tests/test_codecompletion.h

https://commits.kde.org/kdevelop/27cca2f34f407d02212b15deb69d4506ce30d8f0

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to