Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enforce the use of override #78

Merged
merged 2 commits into from
Mar 18, 2024
Merged

Enforce the use of override #78

merged 2 commits into from
Mar 18, 2024

Commits on Jan 17, 2024

  1. Added -Wsuggest-override to the warning list 

    This warning is available on GCC since 5.1 and Clang since 11.0.0.

Since GCC 9.2 it also does not warn if 'final' is used instead of
'override'. This was the case in Clang since the warning was introduced.
    @dk949
    dk949 committed Jan 17, 2024
    Configuration menu
    Copy the full SHA
    19e413d View commit details
    Browse the repository at this point in the history

  2. Moved suggest-override to the GCC only section 

    I did not realise Clang 6 was the earliest supported Clang version.
    @dk949
    dk949 committed Jan 17, 2024
    Configuration menu
    Copy the full SHA
    5befd22 View commit details
    Browse the repository at this point in the history