Fix block argument destructuring with optional parameters#9211
Fix block argument destructuring with optional parameters#9211headius merged 2 commits intojruby:masterfrom
Conversation
2bd046d to
fd6b541
Compare
|
Thanks for looking into this. That's a peculiar issue. |
2b12e64 to
28dd1ae
Compare
28dd1ae to
2f436b1
Compare
|
@headius thanks. I think everything is green now. The single failing test looks like an unrelated time-sensitive issue. |
f0ac831 to
574c519
Compare
|
@evaniainbrooks I think those last failures are due to a similar change being needed in some JIT-related path. That spec run runs the language specs 4 times in the same runtime, so that the JIT can fire and fully bind all calls. I'll see if I can suss out what's wrong. |
|
I think I found the right spot in JIT's argument-handling logic to patch. It has diverged for various reasons and needs to be reined in, but if this passes we can proceed for now. |
|
@headius thank you for the explanation. That certainly explains some other things that perplexed me while I was stepping through this. |
issue #9208