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.

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

fe: Produce an error in case of a declaration in a lazy expression, #1666 #1667

Merged
merged 3 commits into from
Jul 3, 2023

Conversation

fridis
Copy link
Member

@fridis fridis commented Jun 29, 2023

Currently, features declared in an expression used as a lazy value will cause errors since the declaraions are resolved twice, once before and once after they were wrapped in the Lazy instance.

This patch is only a workaround that produces an error message, explains what to do alternatively and what needs to be improved in the Fuzion code to properly fix this.

…1666

Currently, features declared in an expression used as a lazy value will cause
errors since the declaraions are resolved twice, once before and once after they
were wrapped in the Lazy instance.

This patch is only a workaround that produces an error message, explains what to
do alternatively and what needs to be improved in the Fuzion code to properly
fix this.
@fridis fridis requested a review from maxteufel June 29, 2023 15:16
…zy expr

Moved the code for this into the if that does the handling of lazy.
src/dev/flang/ast/AstErrors.java Outdated Show resolved Hide resolved
@maxteufel maxteufel merged commit e1e000d into main Jul 3, 2023
5 checks passed
@maxteufel maxteufel deleted the produce_error_in_case_of_decl_in_lazy branch July 3, 2023 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants