Project

General

Profile

Actions

Feature #14091

closed

Remove Time#succ

Added by pixeltrix (Andrew White) over 6 years ago. Updated over 2 years ago.

Status:
Closed
Target version:
-
[ruby-core:83696]

Description

The method Time#succ was marked as obsolete in Ruby 1.9.3 and whilst it remains Rails has to patch Range#each and Range#step to prevent logs from being filled with the deprecation messages (see https://github.com/rails/rails/pull/11474). It's been over 8 years now which seems like a more than adequate period to wait before removal.


Files

remove-time-succ.diff (3.81 KB) remove-time-succ.diff pixeltrix (Andrew White), 11/07/2017 02:44 PM

Updated by shevegen (Robert A. Heiler) over 6 years ago

Interesting.

If this is true, perhaps ruby core forgot about it.

I had a look at the current docu for Time:

https://ruby-doc.org/core/Time.html

https://ruby-doc.org/core-2.4.2/Time.html#method-i-succ

The documentation reads:

"Returns a new Time object, one second later than time. #succ is obsolete
since 1.9.2 for time is not a discrete value."

So I am inclined to agree with Andrew - either it is indeed obsolete, in
which case it may be time to remove it; and if it is not obsolete, then
the documentation should change, IMO.

(On a side note, perhaps there could be a simple way to ask Ruby which
methods may be deprecated or so... Kernel#deprecated_methods? or something,
but anyway, I should file an own issue request for that; I just don't
know if other people may find that useful. Sorry for side-tracking from
the main issue thread here for a moment!)

Updated by hsbt (Hiroshi SHIBATA) over 6 years ago

  • Status changed from Open to Assigned
  • Assignee set to akr (Akira Tanaka)

Updated by akr (Akira Tanaka) over 4 years ago

  • Assignee changed from akr (Akira Tanaka) to matz (Yukihiro Matsumoto)

It's matz issue.

Updated by jeremyevans0 (Jeremy Evans) over 2 years ago

  • Status changed from Assigned to Closed

Time#succ was removed in Ruby 3.0.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0