Project

General

Profile

Actions

Bug #4401

closed

override :method method breaks Psych

Added by godfat (Lin Jen-Shin) about 13 years ago. Updated almost 13 years ago.

Status:
Closed
Target version:
ruby -v:
ruby 1.9.2p136 (2010-12-25 revision 30365) [x86_64-darwin10.5.0]
Backport:
[ruby-core:35255]

Description

=begin
I am not sure if this could be considered a bug, but it used to work.
To reproduce it:

ruby -r psych -v -e 'Psych.dump(Struct.new(:method).new)'
ruby 1.9.2p136 (2010-12-25 revision 30365) [x86_64-darwin10.5.0]
/usr/local/Cellar/ruby/1.9.2-p136/lib/ruby/1.9.1/psych/visitors/yaml_tree.rb:48:in method': wrong number of arguments(1 for 0) (ArgumentError) from /usr/local/Cellar/ruby/1.9.2-p136/lib/ruby/1.9.1/psych/visitors/yaml_tree.rb:48:in accept'
from /usr/local/Cellar/ruby/1.9.2-p136/lib/ruby/1.9.1/psych/visitors/yaml_tree.rb:36:in <<' from /usr/local/Cellar/ruby/1.9.2-p136/lib/ruby/1.9.1/psych.rb:165:in dump'
from -e:1:in `'

I don't have a good idea how to fix it,
but here's some demonstration:

https://github.com/godfat/ruby/commit/f4e0e8f781b05c767ad2472a43a4ed0727a75708
https://github.com/godfat/ruby/commit/c7a6cf975d88828c2ed27d253f41c480f9b66ad6

Patches attached as well, the same as above two commits.
I think they are against r30885.

Many thanks!

p.s. this issue was found in delayed_job's:
class PerformableMethod < Struct.new(:object, :method, :args)
=end


Files

Actions #1

Updated by godfat (Lin Jen-Shin) about 13 years ago

=begin
The other way, might be:

 Object.instance_method(:method).bind(target).call(:to_yaml).source_location.first

Either way is fine for me.
=end

Actions #2

Updated by headius (Charles Nutter) about 13 years ago

=begin
Fails the same in JRuby (we just reuse Psych's .rb code): http://jira.codehaus.org/browse/JRUBY-5520
=end

Actions #3

Updated by shyouhei (Shyouhei Urabe) almost 13 years ago

  • Status changed from Open to Assigned
Actions #4

Updated by ko1 (Koichi Sasada) almost 13 years ago

Aaron, how about it?

Updated by tenderlovemaking (Aaron Patterson) almost 13 years ago

  • Status changed from Assigned to Closed
  • % Done changed from 0 to 100

This was fixed in r31075. I'm not sure why redmine didn't automatically close.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0