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

Fix issue 18265 #18445

Closed
wants to merge 2 commits into from
Closed

Fix issue 18265 #18445

wants to merge 2 commits into from

Conversation

hamzaremmal
Copy link
Member

@hamzaremmal hamzaremmal commented Aug 22, 2023

Fix how the span is computed for templates.

Fixes #18265

@jan-pieter
Copy link
Contributor

Now it crashes on other tests 😞
java.lang.AssertionError: assertion failed: position error, child positions overlap or in wrong order
So I'm afraid this is not the right solution...

@hamzaremmal hamzaremmal marked this pull request as draft August 22, 2023 17:22
@hamzaremmal
Copy link
Member Author

FYI : I will be back working on this PR this weekend !

i10416 added a commit to i10416/dotty that referenced this pull request Oct 22, 2023
This commit fixes a bug that causes Scala compiler to crash due to position error
when an extension method followed by a newline has neither type annotation nor `= <expr>`.

This commit is based on scala#18445.

Co-Authored-By: @hamzaremmal
@hamzaremmal
Copy link
Member Author

See #18743

@hamzaremmal hamzaremmal closed this Feb 1, 2024
hamzaremmal added a commit that referenced this pull request Feb 1, 2024
close #18265 
This commit fixes a bug that causes Scala compiler to crash due to
position error when an extension method followed by a newline has
neither type annotation nor `= <expr>`.

This commit is based on #18445.

Co-Authored-By: @hamzaremmal
WojciechMazur pushed a commit that referenced this pull request Jun 28, 2024
This commit fixes a bug that causes Scala compiler to crash due to position error
when an extension method followed by a newline has neither type annotation nor `= <expr>`.

This commit is based on #18445.

Co-Authored-By: @hamzaremmal

[Cherry-picked caf30e3]
WojciechMazur pushed a commit that referenced this pull request Jun 30, 2024
This commit fixes a bug that causes Scala compiler to crash due to position error
when an extension method followed by a newline has neither type annotation nor `= <expr>`.

This commit is based on #18445.

Co-Authored-By: @hamzaremmal

[Cherry-picked caf30e3]
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.

Crash on new line after extension def without result type or RHS
2 participants