Skip to content

Fix columnation and forward port improvements #15224

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

Closed
wants to merge 2 commits into from

Conversation

som-snytt
Copy link
Contributor

Avoid boxed ints when creating line index table.
Reject more bad inputs, but keep the convention
that the current line at EOF is the last line
(whether or not the last line is empty).

@som-snytt
Copy link
Contributor Author

Error:  java.lang.IndexOutOfBoundsException: 229675
Error:  	at dotty.tools.dotc.util.SourceFile.offsetToLine(SourceFile.scala:183)
Error:  	at dotty.tools.backend.jvm.BCodeSkelBuilder$PlainSkelBuilder.lineNumber(BCodeSkelBuilder.scala:551)

@som-snytt
Copy link
Contributor Author

Due to disconnect between inlined spans and sources, won't be able to enforce good spans yet.

@som-snytt som-snytt force-pushed the issue/15209 branch 2 times, most recently from 59a917d to c425bc8 Compare May 20, 2022 14:13
som-snytt added 2 commits July 9, 2022 19:50
Avoid boxed ints when creating line index table.
Reject more bad inputs, but keep the convention
that the current line at EOF is the last line
(whether or not the last line is empty).
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.

1 participant