Issue #18237 has been reported by jwmittag (Jrg W Mittag).

----------------------------------------
Bug #18237: Remove unnecessary checks for `Time` in `Range#each` as per the comment
https://bugs.ruby-lang.org/issues/18237

* Author: jwmittag (Jrg W Mittag)
* Status: Open
* Priority: Normal
* ruby -v: ruby 3.1.0dev (2021-10-03T09:41:29Z master bf4f3ef5ad) [x86_64-darwin19]
* Backport: 2.6: UNKNOWN, 2.7: UNKNOWN, 3.0: UNKNOWN
----------------------------------------
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.



-- 
https://bugs.ruby-lang.org/

Unsubscribe: <mailto:ruby-core-request / ruby-lang.org?subject=unsubscribe>
<http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>