Since we can't `RubyVM::InstructionSequence#to_binary` when `Coverage` is running, it is useful to be able to ask ruby if coverage is active. This is possible with `Coverage.peek_result`, but not efficient, since it involves quite a b...burke (Burke Libbey)
Thank you for the feedback! I've attached an updated patch to address the issues. As for testing it, I haven't been able to think of a reasonable method to verify the behaviour without using dtrace/strace, since the only observable ef...burke (Burke Libbey)
All good points, thank you. I definitely agree that the implementation is not good. Feel free to close this. I'll see if I can find a better way to accomplish my goals here some other time.burke (Burke Libbey)
Ah, I was too hasty with the rebase from my 2.3.3 branch. I've attached a fixed patch. Note also that trunk has already eliminated the double-fstat, so this only reduces the number of syscalls when a feature is specified by absolute path...burke (Burke Libbey)
Don't open file twice when specified by absolute path. When invoking `require '/a.rb'` (i.e. via an absolute path), ruby generates this sequence of syscalls: open /a.rb fstat64 /a.rb close /a.rb open /a.r...burke (Burke Libbey)
Hi there, I've attached a patch to add a new `:constant_access` TracePoint event. One feature and/or bug, depending on your perspective: since this event fires from `setinlinecache`, the event is only triggered: - **after nested...burke (Burke Libbey)