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

Constificator: suggest to add const to & parameter in overriding member function #24

Open
PeterSommerlad opened this issue Mar 7, 2019 · 0 comments
Open

Constificator: suggest to add const to & parameter in overriding member function #24

PeterSommerlad opened this issue Mar 7, 2019 · 0 comments
Assignees
@fmorgner
Labels
bug Something isn't working

Comments

@PeterSommerlad
Copy link

Expected Behavior

constificator should not suggest to add const to a reference parameter in a virtual or overriding member function. at least not in a derived class, where changing the signature breaks overriding

Actual Behavior

see above

Cevelop Version, Operating System and Compiler

current
@fmorgner fmorgner transferred this issue from Cevelop/Issues Feb 23, 2021
@fmorgner fmorgner added the bug Something isn't working label Feb 23, 2021
@fmorgner fmorgner self-assigned this Feb 23, 2021
@fmorgner fmorgner changed the title constificator suggest to add const to & parameter in overriding member function Constificator: suggest to add const to & parameter in overriding member function Feb 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fmorgner fmorgner

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fmorgner @PeterSommerlad