[GROOVY-9373] ASM: rework line numbers for blocks with fast-path, try/catch or return statement insertion

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.4.18, 2.5.9, 3.0.0-rc-3
• Fix Version/s: 4.0.0-alpha-3
• Assignee: Eric Milles
• Reporter: Eric Milles
• Created: 22/Jan/20 19:50
• Updated: 02/May/21 09:18
• Resolved: 02/Mar/21 15:18

Description

GROOVY-4063, GROOVY-4505, GROOVY-7647, GROOVY-8742, GROOVY-9076, GROOVY-9126 and GROOVY-9199 present (possibly) conflicting concerns. Code coverage metrics are thrown off when line number information exists outside of the reachable paths in the bytecode. Debug break and step exhibit strange behaviors when the bytecode contains insufficient or duplicated line number information.

Can the ASM changes in StatementWriter be reworked to address these concerns?

See also https://github.com/jacoco/jacoco/issues/884