Re: [jruby-user] JRuby head: splat (i.e., **args) method params?

Ah yeah.  We are not delivering keyword arguments to zsuper.  Thanks for
the simple reproduction.  I made an issue for it:

https://github.com/jruby/jruby/issues/1943

-Tom



On Tue, Sep 2, 2014 at 11:45 PM, Josep Blanquer <[email protected]>
wrote:

> Aha! intersting...yes I can...so it seems a consequence of the "super"
> method.
> Here you have a repro:
>
> ------------------------------------------------------------
> class Foo
>   def initialize(name, context,**opts)
>     p opts
>   end
> end
>
> class Bar < Foo
>     def initialize(name, context, path: nil)
>       super
>     end
> end
>
> Bar.new(1, 2)
> Bar.new(1,2, path: "there")
> ------------------------------------------------------------
>
> bug:foo $ rvm 2.1
> bug:foo $ ruby /tmp/x.rb
> {:path=>nil}
> {:path=>"there"}
> bug:foo $ rvm jruby-head
> bug:foo $ ruby /tmp/x.rb
> ArgumentError: wrong number of arguments (3 for 2)
>   initialize at /tmp/x.rb:9
>   initialize at /tmp/x.rb:9
>       (root) at /tmp/x.rb:13
>
> while in MRI ruby both calls work (with and without the hash), in jruby
> both fail with the arguments error.
> So, something about the super call doesn't quite work as MRI does.
>
> Cheers,
>
> Josep M.
>
>
> On Tue, Sep 2, 2014 at 3:50 PM, Thomas E Enebo <[email protected]>
> wrote:
>
>> Can you give a more complete example.  This works for me:
>>
>> class Foo
>>   def initialize(name, context,**opts)
>>     p opts
>>   end
>> end
>>
>> Foo.new(1, 2, a: 1)
>>
>> ./bin/jruby -v snippets/kw15.rb
>> jruby 9000.dev-SNAPSHOT (2.1.2p142) 2014-09-02 8771d02 Java HotSpot(TM)
>> 64-Bit Server VM 24.60-b09 on 1.7.0_60-b19 [darwin-x86_64]
>> /Users/enebo/work/jruby/lib/ruby/shared/rubygems/defaults/jruby.rb:92
>> warning: Ambiguous first argument; make sure.
>> /Users/enebo/work/jruby/lib/ruby/shared/rubygems/defaults/jruby.rb:96
>> warning: shadowing outer local variable - dir
>> {:a=>1}
>>
>>
>> On Tue, Sep 2, 2014 at 4:52 PM, Josep Blanquer <[email protected]>
>> wrote:
>>
>>> Hello,
>>>
>>>  It seems that as of the latest head from today:
>>> jruby 9000.dev-SNAPSHOT (2.1.2p142) 2014-09-02 8771d02 Java HotSpot(TM)
>>> 64-Bit Server VM 24.0-b56 on 1.7.0_40-b43 [darwin-x86_64]
>>>
>>> The following 2.1 syntax is not implemented?
>>>
>>> ArgumentError: wrong number of arguments (3 for 2)
>>>                          initialize at /...../file.rb:11
>>>
>>> Where that line is:
>>>
>>>     def initialize(name, context,**opts)
>>>
>>>
>>> am I missing something to make that happen in the current codebase? or
>>> is this a well know feature that still has to be implemented?
>>>
>>> If the latter is true: do you have any inkling as to when that could be
>>> done?
>>>
>>> Thank you!
>>>
>>> Josep M.
>>>
>>
>>
>>
>> --
>> blog: http://blog.enebo.com       twitter: tom_enebo
>> mail: [email protected]
>>
>
>


-- 
blog: http://blog.enebo.com       twitter: tom_enebo
mail: [email protected]