diff options
| author | tenderlove <tenderlove@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2010-03-29 23:57:25 +0000 |
|---|---|---|
| committer | tenderlove <tenderlove@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2010-03-29 23:57:25 +0000 |
| commit | 797f482fe89cf9af1467fc0e7f0707d8c5e4141f (patch) | |
| tree | 6b83b7cebe41aa848f237fed80ed8b0d626bc5fd | |
| parent | c294fcc0abe83bae433bc08b009db1d03b39e864 (diff) | |
* lib/psych.rb: Fix problem with empty and white-space only strings.
Thanks Peter McLain!
* test/psych/test_psych.rb: tests for change.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@27099 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
| -rw-r--r-- | ChangeLog | 6 | ||||
| -rw-r--r-- | lib/psych.rb | 6 | ||||
| -rw-r--r-- | test/psych/test_psych.rb | 7 |
3 files changed, 17 insertions, 2 deletions
@@ -1,3 +1,9 @@ +Tue Mar 30 08:10:59 2010 Aaron Patterson <aaron@tenderlovemaking.com> + + * lib/psych.rb: Fix problem with empty and white-space only strings. + Thanks Peter McLain! + * test/psych/test_psych.rb: tests for change. + Tue Mar 30 05:31:39 2010 Aaron Patterson <aaron@tenderlovemaking.com> * lib/psych.rb: documentation updates. Thanks Peter McLain! diff --git a/lib/psych.rb b/lib/psych.rb index 574bc7e641..c14259373a 100644 --- a/lib/psych.rb +++ b/lib/psych.rb @@ -101,7 +101,8 @@ module Psych # Psych.load("--- a") # => 'a' # Psych.load("---\n - a\n - b") # => ['a', 'b'] def self.load yaml - parse(yaml).to_ruby + result = parse(yaml) + result ? result.to_ruby : result end ### @@ -113,7 +114,8 @@ module Psych # # See Psych::Nodes for more information about YAML AST. def self.parse yaml - parse_stream(yaml).children.first.children.first + children = parse_stream(yaml).children + children.empty? ? false : children.first.children.first end ### diff --git a/test/psych/test_psych.rb b/test/psych/test_psych.rb index fce070fa39..88fe83c19a 100644 --- a/test/psych/test_psych.rb +++ b/test/psych/test_psych.rb @@ -66,4 +66,11 @@ class TestPsych < Psych::TestCase assert_equal 'hello world', Psych.parse_file(name).transform end + + def test_degenerate_strings + assert_equal false, Psych.load(' ') + assert_equal false, Psych.parse(' ') + assert_equal false, Psych.load('') + assert_equal false, Psych.parse('') + end end |
