Actions
Bug #16339
closedForwardable#def_delegator warns and is incorrect on trunk when passed keyword arguments
Status:
Closed
Assignee:
-
Target version:
-
ruby -v:
ruby 2.7.0dev (2019-11-10T05:56:38Z master 4570284ce1) [x86_64-linux]
Backport:
Description
Because it uses *args, &block
-style delegation.
require 'forwardable'
class C
extend Forwardable
def_delegator :self, :target, :delegate
def target(*args, **kwargs)
[args, kwargs]
end
end
p C.new.target(1, b: 2) # => [[1], {:b=>2}]
p C.new.delegate(1, b: 2)
# ruby-trunk/lib/ruby/2.7.0/forwardable.rb:231: warning: The last argument is used as the keyword parameter
# del.rb:6: warning: for `target' defined here
# => [[1], {:b=>2}]
p C.new.target({}, **{}) # => [[{}], {}]
p C.new.delegate({}, **{})
# ruby-trunk/lib/ruby/2.7.0/forwardable.rb:231: warning: The last argument is used as the keyword parameter
# del.rb:6: warning: for `target' defined here
# => [[], {}]
Which also illustrates we're missing important tests/specs for Forwardable.
Actions
Like0
Like0Like0Like0Like0