Bug #15332: coverage and InstructionSequence regression

• Status: Closed
• Assignee: -
• Target version: 2.6
• Backport: 2.3: UNKNOWN, 2.4: UNKNOWN, 2.5: UNKNOWN

[marcandre (Marc-Andre Lafortune)]

With the current head:

    require 'coverage'
    Coverage.start
    RubyVM::InstructionSequence.compile("puts 'hi'", 'hello_world.rb').eval
    Coverage.result # => {}, should be {'hello_world.rb' => [1]}

This is not intended, right?

[mame (Yusuke Endoh)]

The change itself is actually not intended.
But I'm unsure if it should handle ISeq-loaded code as a measuring target, because currently eval-loaded code is not a target:

require 'coverage'
Coverage.start
eval("puts 'hi'", binding, "hello_world.rb")
p Coverage.result # => {} in 2.5 and trunk

Do you have actual use case? (Is it needed for bootsnap or so?) If not, I'd like to add it into NEWS as a subtle incompatibility.

[marcandre (Marc-Andre Lafortune)]

I don't have an important use case, no.

[naruse (Yui NARUSE)]

• Status changed from Open to Closed