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

Incorrect startPosition of a SingleVariableDeclaration in the catch block #2896

Conversation

subyssurendran666
Copy link
Contributor

@subyssurendran666 subyssurendran666 commented Sep 3, 2024

What it does

#2564

How to test

Author checklist

@subyssurendran666 subyssurendran666 force-pushed the incorrect-startPosition-of-a-SingleVariableDeclaration-in-the-catch-block branch 2 times, most recently from 9b73834 to e45dec2 Compare September 23, 2024 05:41
@subyssurendran666 subyssurendran666 force-pushed the incorrect-startPosition-of-a-SingleVariableDeclaration-in-the-catch-block branch 2 times, most recently from 9ceb866 to dd6a1bb Compare September 25, 2024 06:46
@subyssurendran666 subyssurendran666 force-pushed the incorrect-startPosition-of-a-SingleVariableDeclaration-in-the-catch-block branch 2 times, most recently from df3f8ba to 8cb4db4 Compare September 25, 2024 08:21
@jarthana
Copy link
Member

@subyssurendran666 Sorry for input in a piecemeal manner. Can you also add a testcase with a comment on the catch parameter? Like this:
catch(/** abc*/ Exception e) {}
We should test that the source positions includes the comment.

@subyssurendran666 subyssurendran666 force-pushed the incorrect-startPosition-of-a-SingleVariableDeclaration-in-the-catch-block branch from 8cb4db4 to 98533cf Compare September 26, 2024 06:29
@subyssurendran666
Copy link
Contributor Author

@subyssurendran666 Sorry for input in a piecemeal manner. Can you also add a testcase with a comment on the catch parameter? Like this: catch(/** abc*/ Exception e) {} We should test that the source positions includes the comment.

@jarthana I have added a new test case. Could you please review my changes ?

@subyssurendran666 subyssurendran666 force-pushed the incorrect-startPosition-of-a-SingleVariableDeclaration-in-the-catch-block branch from 98533cf to 8d66756 Compare September 30, 2024 03:32
@jarthana jarthana merged commit 4833c76 into eclipse-jdt:master Sep 30, 2024
9 checks passed
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.

2 participants