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 the following pull requests for details: 
 * [YARV GitHub [GitHub Pull Request 4928](https://github.com/ruby/ruby/pull/4928) 
 * [ruby/spec GitHub Pull Request 852](https://github.com/ruby/spec/pull/852) 

 Patches are for details. Patch is also attached.

Back