I discussed this in detail with @mame and @matz. @mame (to reply here too) what I meant is using Prism with the `Method|UnboundMethod|Proc` and the `source_range` to find the Prism node, and then compute the max end offset by walking th...Eregon (Benoit Daloze)
I discussed this in detail with @mame and @matz. One observation from me is the addition of `Ruby::Node` (which duplicates a very large part of the Prism Ruby API, >100 classes) is mostly motivated from using `node_id`, because reusin...Eregon (Benoit Daloze)
At the meeting @matz said he was positive that `deep_freeze` is "freeze but recursively". More precisely: * `String.deep_freeze` => freezes `String` like `String.freeze` but not anything further like `Object` or so. (This might unex...Eregon (Benoit Daloze)
zenspider (Ryan Davis) wrote in #note-5: > so what should `rescue /regexp/` do? It would do `/regexp/ === exception`, which is not useful but consistent. The `assert_raises` example is interesting, I recall https://github.com/test...Eregon (Benoit Daloze)
I think a better fix, related to this issue, would be to stop checking the class of rescue clauses, I think any expression should be allowed and just call `===` on them. So one could do e.g.: ```ruby begin raise "nope" rescue ->...Eregon (Benoit Daloze)
The reason is `rescue` clauses are evaluated lazily, and stop at the first matching one. I believe this is by design for efficiency. Evaluating every `rescue` clause when not necessary would be some overhead and have potentially unwant...Eregon (Benoit Daloze)
Thank you for reviewing this ticket. I agree with both points. I thought it would be nice-to-have to also have this for classes defined in Ruby but as you say the worst case is NoMethodError, and the specifics of the proposal are mostly...Eregon (Benoit Daloze)
For the start position I think either is fine. I think from the `p` of `proc` is more useful because it gives extra context to what the block is given (a block can't exist on its own, it's always part of something bigger syntax-wise). ...Eregon (Benoit Daloze)