Conversation
|
I test it locally and it fix my problem from this issue |
|
can you merge this fix? |
org.jacoco.core/src/org/jacoco/core/internal/instr/ProbeInserter.java
Outdated
Show resolved
Hide resolved
unfortunately not yet - see #893 (comment) |
|
@Godin Given the complexity of inserting the probe array somewhere "in the middle" we might consider simply adding it to the end. This will increase the byte code size because it will be less likely that |
|
@marchof regarding #893 (comment)
I'm not so sure, are you? 😉
And method size is limited 😉
I wouldn't call it "simply" - |
ae8f2f2 to
a74404c
Compare
|
@marchof I rebased this PR and fixed the problem described in #893 (comment), it still lacks in-code comments requested in #893 (comment) and the changelog entry, but maybe you already can review what else needs to be tweaked in the implementation? Fun fact: the issue can be reproduced by using Kotlin compiler versions starting from released 7 years ago |
org.jacoco.core.test/src/org/jacoco/core/internal/instr/SafetySlotTest.java
Outdated
Show resolved
Hide resolved
org.jacoco.core.test/src/org/jacoco/core/internal/instr/SafetySlotTest.java
Outdated
Show resolved
Hide resolved
org.jacoco.core.test/src/org/jacoco/core/internal/instr/SafetySlotTest.java
Outdated
Show resolved
Hide resolved
|
@marchof I added comments and updated changelog, so IMO this can be merged 🎉 |
Fixes #767
Fixes #1412
Fixes #1429