Skip to content

Don't double-include line and column information in SyntaxError (fixes #5357)#5409

Open
MatrixFrog wants to merge 1 commit into
boa-dev:mainfrom
MatrixFrog:issue5357
Open

Don't double-include line and column information in SyntaxError (fixes #5357)#5409
MatrixFrog wants to merge 1 commit into
boa-dev:mainfrom
MatrixFrog:issue5357

Conversation

@MatrixFrog

@MatrixFrog MatrixFrog commented Jun 20, 2026

Copy link
Copy Markdown

This Pull Request fixes #5357

The SyntaxError includes the location information already so it doesn't need to add the line and column number in the error message as well.

@MatrixFrog MatrixFrog requested a review from a team as a code owner June 20, 2026 19:21
@github-actions github-actions Bot added the Waiting On Review Waiting on reviews from the maintainers label Jun 20, 2026
@github-actions github-actions Bot added this to the v1.0.0 milestone Jun 20, 2026
@github-actions github-actions Bot added C-Parser Issues surrounding the parser C-Tests Issues and PRs related to the tests. labels Jun 20, 2026
@MatrixFrog MatrixFrog changed the title Proposed fix for #5357 Don't double-include line and column information in SyntaxError (fixes #5357) Jun 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-Parser Issues surrounding the parser C-Tests Issues and PRs related to the tests. Waiting On Review Waiting on reviews from the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Syntax error mentions location twice

1 participant