From: "ioquatix (Samuel Williams)" <noreply@...>
Date: 2022-10-31T07:04:49+00:00
Subject: [ruby-core:110558] [Ruby master Feature#19094] `sleep(nil)` vs `sleep()` and replicating the default implementation.

Issue #19094 has been updated by ioquatix (Samuel Williams).


By the way, my motivation was to make this easier for the user: https://github.com/socketry/async/issues/161

----------------------------------------
Feature #19094: `sleep(nil)` vs `sleep()` and replicating the default implementation.
https://bugs.ruby-lang.org/issues/19094#change-99887

* Author: ioquatix (Samuel Williams)
* Status: Open
* Priority: Normal
* Assignee: ioquatix (Samuel Williams)
----------------------------------------
```
> sleep(nil)
(irb):1:in `sleep': can't convert NilClass into time interval (TypeError)
```

However, I feel that this makes implementing a compatible sleep method a little difficult.

```ruby
def sleep(time = nil)
  if time
    sleep_some(time)
  else
    sleep_forever
  end
end
```

Can we consider allowing `sleep(nil)` and `sleep(false)` to be the same as `sleep()` to simplify this behaviour?

Otherwise, it's hard to proxy (e.g. fiber scheduler, or even just a plain old variable argument).

e.g.

```ruby
class Sleeper
  def initialize(time = nil)
    @time = time
  end

  def sleep
    Kernel::sleep(@time) # Hard to get the behaviour of `sleep()` here without explicitly handling/checking.
  end
end
```




-- 
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>