Codecov Report

Merging #4198 (09ac1d7) into main (c5850c4) will increase coverage by 0.86%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #4198      +/-   ##
============================================
+ Coverage     83.43%   84.30%   +0.86%     
- Complexity     3149     3236      +87     
============================================
  Files           456      468      +12     
  Lines          9017    10185    +1168     
  Branches       1754     1786      +32     
============================================
+ Hits           7523     8586    +1063     
- Misses          567      659      +92     
- Partials        927      940      +13     

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c5850c4...09ac1d7. Read the comment docs.

LGTM 👍 Could you add also the test you mentioned here: #4197 (comment) So we could technically close #4197 till we have a new reproducer.

I can, but the intent of the test I added was to reflect that use case in a clearer/descriptive way.

I see nothing different that happens in the provided source code, so in a way I think that test would be superfluous. I did test it to make sure, and it passed.

Let me know what you prefer:

• Keep it as is.
• Add it as a new test.
• Replace """fun f() { val ex = Exception() }""" with the code reported in the issue, but keep the test name/a single test.