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

CFE should not allow redirection from const to non-const constructor #34161

Closed
askeksa-google opened this issue Aug 16, 2018 · 2 comments
Closed

CFE should not allow redirection from const to non-const constructor #34161

askeksa-google opened this issue Aug 16, 2018 · 2 comments
Assignees
@dhil
Labels
area-front-end Use area-front-end for front end / CFE / kernel format related issues. front-end-fasta front-end-missing-error

Comments

@askeksa-google
Copy link

CFE version of #27617

Caught by language_2/regress_27617_test.
@askeksa-google askeksa-google added front-end-fasta area-front-end Use area-front-end for front end / CFE / kernel format related issues. front-end-missing-error labels Aug 16, 2018
@askeksa-google askeksa-google changed the title Redirection from const to non-const constructor allowed CFE should not allow redirection from const to non-const constructor Aug 20, 2018
@askeksa-google
Copy link
Author

The same goes for factory constructors. See co19_2/Language/Classes/Constructors/Factories/const_modifier_t01 and co19_2/Language/Classes/Constructors/Factories/const_modifier_t02.

@dhil dhil self-assigned this Aug 21, 2018
@dhil dhil reopened this Aug 23, 2018
@dhil
Copy link

dhil commented Aug 23, 2018

Reopening, as it was unintentionally closed by the first CL. I will keep it open until I have landed the second CL that implements the check for factory constructors.

@askeksa-google askeksa-google mentioned this issue Sep 28, 2018
Closed
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dhil dhil

Labels
area-front-end Use area-front-end for front end / CFE / kernel format related issues. front-end-fasta front-end-missing-error
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dhil @askeksa-google