Project

General

Profile

Actions

Bug #7664

closed

Keyword arguments with `**` make code targeting 1.9 / 2.0 difficult

Added by wycats (Yehuda Katz) almost 12 years ago. Updated almost 12 years ago.

Status:
Rejected
Assignee:
-
Target version:
-
ruby -v:
2.0.0dev
Backport:
[ruby-core:51271]

Description

You could imagine writing code that calls into a user-supplied hook:

log("App booted", level: :info)

It is possible for both Ruby 1.9 and Ruby 2.0 users to write such a hook:

Ruby 1.9

def log(string, options={})
level = options[:level]
puts "#{level}: #{string}"
end

Ruby 2.0

def log(string, level: nil)
puts "#{level}: #{string}"
end

So far so good. It's also possible for Ruby 2.0 users to handle arbitrary arguments:

Ruby 2.0

def log(string, **options)
# pass options along to other methods without explicit keyword args support
end

However, it is not possible to call into methods with arbitrary arguments in a compatible way:

Ruby 1.9

args = [ string ] + [ options ]
log(*args)

Ruby 2.0

log(string, **options)

Unless I'm missing something, this makes it impossible to write code that targets both Ruby 1.9 and Ruby 2.0 if you need to send arbitrary keyword arguments to a method.

Also, because ** is new syntax, the only way to handle this problem is via eval:

if RUBY_VERSION > "2"
eval "log(string, **options)", FILE, LINE
else
args = [ string ] + [ options ]
log(*args)
end

This can work if you use class_eval to create two different versions of the method at class creation time, but it's pretty ugly even in that case.

Is there a way to make it possible to have a backwards-compatible calling signature that can target both the Ruby 1.9 and Ruby 2.0 method signatures for doing keyword arguments?

Updated by matz (Yukihiro Matsumoto) almost 12 years ago

  • Status changed from Open to Rejected

I understand your concern. But I consider this as a driving force to jump in to 2.0.
Since we have small compatibility issues between 1.9 and 2.0, I'd suggest moving forward.

Matz.

Updated by nobu (Nobuyoshi Nakada) almost 12 years ago

=begin
Have you tried this in 2.0 actually?

args = [ string ] + [ options ]
log(*args)
=end

Actions

Also available in: Atom PDF

Like0
Like0Like0