ruby-changes:21335
From: tenderlove <ko1@a...>
Date: Tue, 4 Oct 2011 06:32:08 +0900 (JST)
Subject: [ruby-changes:21335] tenderlove:r33384 (trunk): * ext/psych/lib/psych.rb: calling `yaml` rather than `to_yaml`.
tenderlove 2011-10-04 06:31:43 +0900 (Tue, 04 Oct 2011) New Revision: 33384 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=33384 Log: * ext/psych/lib/psych.rb: calling `yaml` rather than `to_yaml`. * ext/psych/lib/psych/nodes/node.rb: Rename `to_yaml` to just `yaml` in order to avoid YAML::ENGINE switching from replacing this method. * test/psych/helper.rb: fix tests for method name change. * test/psych/test_document.rb: ditto * test/psych/visitors/test_emitter.rb: ditto Modified files: trunk/ChangeLog trunk/ext/psych/lib/psych/nodes/node.rb trunk/ext/psych/lib/psych.rb trunk/test/psych/helper.rb trunk/test/psych/test_document.rb trunk/test/psych/visitors/test_emitter.rb Index: ChangeLog =================================================================== --- ChangeLog (revision 33383) +++ ChangeLog (revision 33384) @@ -1,3 +1,12 @@ +Tue Oct 4 06:29:55 2011 Aaron Patterson <aaron@t...> + + * ext/psych/lib/psych.rb: calling `yaml` rather than `to_yaml`. + * ext/psych/lib/psych/nodes/node.rb: Rename `to_yaml` to just `yaml` + in order to avoid YAML::ENGINE switching from replacing this method. + * test/psych/helper.rb: fix tests for method name change. + * test/psych/test_document.rb: ditto + * test/psych/visitors/test_emitter.rb: ditto + Tue Oct 4 06:20:19 2011 Aaron Patterson <aaron@t...> * ext/psych/lib/psych/scalar_scanner.rb: Match values against the Index: ext/psych/lib/psych/nodes/node.rb =================================================================== --- ext/psych/lib/psych/nodes/node.rb (revision 33383) +++ ext/psych/lib/psych/nodes/node.rb (revision 33384) @@ -40,13 +40,14 @@ # Convert this node to YAML. # # See also Psych::Visitors::Emitter - def to_yaml io = nil, options = {} + def yaml io = nil, options = {} real_io = io || StringIO.new(''.encode('utf-8')) Visitors::Emitter.new(real_io, options).accept self return real_io.string unless io io end + alias :to_yaml :yaml end end end Index: ext/psych/lib/psych.rb =================================================================== --- ext/psych/lib/psych.rb (revision 33383) +++ ext/psych/lib/psych.rb (revision 33384) @@ -190,7 +190,7 @@ visitor = Psych::Visitors::YAMLTree.new options visitor << o - visitor.tree.to_yaml io, options + visitor.tree.yaml io, options end ### @@ -204,7 +204,7 @@ objects.each do |o| visitor << o end - visitor.tree.to_yaml + visitor.tree.yaml end ### @@ -212,7 +212,7 @@ def self.to_json o visitor = Psych::Visitors::JSONTree.new visitor << o - visitor.tree.to_yaml + visitor.tree.yaml end ### Index: test/psych/test_document.rb =================================================================== --- test/psych/test_document.rb (revision 33383) +++ test/psych/test_document.rb (revision 33384) @@ -18,12 +18,12 @@ end def test_emit_tag - assert_match('%TAG ! tag:tenderlovemaking.com,2009:', @stream.to_yaml) + assert_match('%TAG ! tag:tenderlovemaking.com,2009:', @stream.yaml) end def test_emit_multitag @doc.tag_directives << ['!!', 'foo.com,2009:'] - yaml = @stream.to_yaml + yaml = @stream.yaml assert_match('%TAG ! tag:tenderlovemaking.com,2009:', yaml) assert_match('%TAG !! foo.com,2009:', yaml) end @@ -31,7 +31,7 @@ def test_emit_bad_tag assert_raises(RuntimeError) do @doc.tag_directives = [['!']] - @stream.to_yaml + @stream.yaml end end @@ -40,7 +40,7 @@ end def test_emit_version - assert_match('%YAML 1.1', @stream.to_yaml) + assert_match('%YAML 1.1', @stream.yaml) end end end Index: test/psych/helper.rb =================================================================== --- test/psych/helper.rb (revision 33383) +++ test/psych/helper.rb (revision 33384) @@ -33,7 +33,7 @@ def assert_cycle( obj ) v = Visitors::YAMLTree.new v << obj - assert_equal(obj, Psych.load(v.tree.to_yaml)) + assert_equal(obj, Psych.load(v.tree.yaml)) assert_equal( obj, Psych::load(Psych.dump(obj))) assert_equal( obj, Psych::load( obj.psych_to_yaml ) ) end Index: test/psych/visitors/test_emitter.rb =================================================================== --- test/psych/visitors/test_emitter.rb (revision 33383) +++ test/psych/visitors/test_emitter.rb (revision 33384) @@ -46,7 +46,7 @@ @visitor.accept s assert_match(/1.1/, @io.string) - assert_equal @io.string, s.to_yaml + assert_equal @io.string, s.yaml end def test_document_implicit_end @@ -61,8 +61,8 @@ @visitor.accept s assert_match(/key: value/, @io.string) - assert_equal @io.string, s.to_yaml - assert(/\.\.\./ !~ s.to_yaml) + assert_equal @io.string, s.yaml + assert(/\.\.\./ !~ s.yaml) end def test_scalar @@ -76,7 +76,7 @@ @visitor.accept s assert_match(/hello/, @io.string) - assert_equal @io.string, s.to_yaml + assert_equal @io.string, s.yaml end def test_scalar_with_tag @@ -91,7 +91,7 @@ assert_match(/str/, @io.string) assert_match(/hello/, @io.string) - assert_equal @io.string, s.to_yaml + assert_equal @io.string, s.yaml end def test_sequence @@ -107,7 +107,7 @@ @visitor.accept s assert_match(/- hello/, @io.string) - assert_equal @io.string, s.to_yaml + assert_equal @io.string, s.yaml end def test_mapping @@ -122,7 +122,7 @@ @visitor.accept s assert_match(/key: value/, @io.string) - assert_equal @io.string, s.to_yaml + assert_equal @io.string, s.yaml end def test_alias @@ -137,7 +137,7 @@ @visitor.accept s assert_match(/&A key: \*A/, @io.string) - assert_equal @io.string, s.to_yaml + assert_equal @io.string, s.yaml end end end -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/