Project

General

Profile

Actions

Bug #15539

closed

Proc.new with no block shouldn't always warn

Added by tenderlovemaking (Aaron Patterson) almost 6 years ago. Updated almost 6 years ago.

Status:
Rejected
Assignee:
-
Target version:
-
[ruby-core:91109]

Description

Hi,

r66772 introduced a warning for the following code:

def foo bar = Proc.new
  bar.call
end

foo      { p "block" } # warn
foo ->() { p "block" } # no warn

I don't think this case of Proc.new should warn. To eliminate warnings, I have to rewrite as:

def foo bar = nil
  if bar
    bar.call
  else
    yield
  end
end

Rails uses this "Proc.new" trick here:

I can change Rails, but I want to know why and I don't see any discussion of r66772 (the commit doesn't have a feature number).

Thanks!

Updated by ko1 (Koichi Sasada) almost 6 years ago

I'll file the reason soon.

Anyway, I recommend you to rewrite it with a block parameter:

def foo bar = nil, &b
  (b || bar).call
end

# or
def foo bar = nil, &b
  bar ||= b
  bar.call
end

Updated by tenderlovemaking (Aaron Patterson) almost 6 years ago

ko1 (Koichi Sasada) wrote:

I'll file the reason soon.

Anyway, I recommend you to rewrite it with a block parameter:

def foo bar = nil, &b
  (b || bar).call
end

# or
def foo bar = nil, &b
  bar ||= b
  bar.call
end

Ok, no problem! I'll update Rails and reference the feature when posted. 😊

Updated by jeremyevans0 (Jeremy Evans) almost 6 years ago

ko1 (Koichi Sasada) wrote:

I'll file the reason soon.

Anyway, I recommend you to rewrite it with a block parameter:

def foo bar = nil, &b
  (b || bar).call
end

I also ran into this issue as Proc.new was used in a few places in Sequel. Removing implicit block support for Proc.new makes it no longer possible to have a default value for a positional argument that uses the block, since the block argument variable is not set until after default values for positional arguments (e.g. https://github.com/jeremyevans/sequel/commit/34faf32c6eddc02d1499748a431f3fdf46bfc5a4#diff-29540b6b617d51d3f5356b1cdfc534b0L61). I feel the loss of functionality is acceptable, since Proc.new's implicit block support was fairly magical and not widely known/used.

Updated by ko1 (Koichi Sasada) almost 6 years ago

  • Status changed from Open to Rejected

makes it no longer possible to have a default value for a positional argument that uses the block

Surprisingly, we can:

def foo o = binding.local_variable_get(:b), &b
  p o.class
end

foo{} #=> Proc

(of course it is joke code. it is too implementation dependent code so we shouldn't rely on this behavior)

Anyway, I close this ticket and discuss more on new ticket I'll file.

Thanks,
Koichi

Updated by Eregon (Benoit Daloze) almost 6 years ago

I always thought Proc.new working without an explicit block was an unintended feature in MRI, and indeed fairly magical, so I support taking a more explicit way in 2.7.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0