Project

General

Profile

Bug #18237

Updated by jwmittag (Jörg W Mittag) about 3 years ago

In commit:7817a438eb1803e7b3358f43bd1f38479badfbdc, the implementation of `Time#succ`, which had been deprecated for 10 years, was finally removed. 

 During that time, there was an explicit `instance_of?` check in source:range.c#L350 with a comment that the check should be removed once `Time#succ` is removed. 

 Since `Time#succ` is now gone, I propose removing this check. 

 Note: this should be coordinated with adding a version guard to the corresponding check in ruby/spec as well. 

 See [GitHub Pull Request 4928](https://github.com/ruby/ruby/pull/4928) 4928](https://github.com/ruby/pull/4928) for details. Patch is also attached.

Back