Skip to content

GROOVY-9373 (part 1): rework line numbers for statements#1443

Merged
daniellansun merged 1 commit intomasterfrom
GROOVY-9373
Dec 16, 2020
Merged

GROOVY-9373 (part 1): rework line numbers for statements#1443
daniellansun merged 1 commit intomasterfrom
GROOVY-9373

Conversation

@eric-milles
Copy link
Copy Markdown
Member

@eric-milles eric-milles commented Dec 14, 2020

https://issues.apache.org/jira/browse/GROOVY-9373
https://issues.apache.org/jira/browse/GROOVY-9199
https://issues.apache.org/jira/browse/GROOVY-9126
https://issues.apache.org/jira/browse/GROOVY-7647

"do" and "try" should not generate line numbers
instructions after last source line are tied to closing brace
instructions before first source line are tied to opening brace

Writing two LINENUMBER instructions in a row is an issue. Given this sequence from groovy.log.classgen system property:

L1
  LINENUMBER 7 L1
L2
  LINENUMBER 8 L2

If user sets breakpoint on line 8 of source, debugger will not stop because it associates line 7 with whatever instructions come next. One example of this is try { on line 7 and some statement on line 8 (see GROOVY-9199).

GROOVY-7647, GROOVY-9126, GROOVY-9199

"do" and "try" should not generate line numbers
instructions after last source line are tied to closing brace
instructions before first source line are tied to opening brace
@daniellansun daniellansun merged commit 703b644 into master Dec 16, 2020
@daniellansun
Copy link
Copy Markdown
Contributor

Merged. Thanks!

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