diff options
author | tenderlove <tenderlove@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-10-22 21:25:02 +0000 |
---|---|---|
committer | tenderlove <tenderlove@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-10-22 21:25:02 +0000 |
commit | 3c73f44c7f779fe9eea823457c4f288aa21d3c32 (patch) | |
tree | dd8b7e9af8979d1738d10ecc4ddde5d5270a9638 | |
parent | 6c6d4568e8b8c8667c0b57ca6e1b5ba5d7acbbe3 (diff) | |
download | ruby-3c73f44c7f779fe9eea823457c4f288aa21d3c32.tar.gz |
* ext/psych/parser.c: just get the constant defined in Ruby.
* ext/psych/lib/psych/syntax_error.rb: Psych::SyntaxError now inherits
from StandardError rather than SyntaxError. Thanks Eric Hodel!
* test/psych/test_exception.rb: tests for change.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@37292 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 9 | ||||
-rw-r--r-- | ext/psych/lib/psych/syntax_error.rb | 5 | ||||
-rw-r--r-- | ext/psych/parser.c | 2 | ||||
-rw-r--r-- | test/psych/test_exception.rb | 21 |
4 files changed, 35 insertions, 2 deletions
@@ -1,3 +1,12 @@ +Tue Oct 23 06:21:05 2012 Aaron Patterson <aaron@tenderlovemaking.com> + + * ext/psych/parser.c: just get the constant defined in Ruby. + + * ext/psych/lib/psych/syntax_error.rb: Psych::SyntaxError now inherits + from StandardError rather than SyntaxError. Thanks Eric Hodel! + + * test/psych/test_exception.rb: tests for change. + Tue Oct 23 06:17:36 2012 Aaron Patterson <aaron@tenderlovemaking.com> * ext/psych/lib/psych/scalar_scanner.rb: Cache symbols while diff --git a/ext/psych/lib/psych/syntax_error.rb b/ext/psych/lib/psych/syntax_error.rb index f79743dba4..f972256f9e 100644 --- a/ext/psych/lib/psych/syntax_error.rb +++ b/ext/psych/lib/psych/syntax_error.rb @@ -1,5 +1,8 @@ module Psych - class SyntaxError < ::SyntaxError + class Error < RuntimeError + end + + class SyntaxError < Error attr_reader :file, :line, :column, :offset, :problem, :context def initialize file, line, col, offset, problem, context diff --git a/ext/psych/parser.c b/ext/psych/parser.c index 0908a1b49f..8c65ce1307 100644 --- a/ext/psych/parser.c +++ b/ext/psych/parser.c @@ -557,7 +557,7 @@ void Init_psych_parser() rb_define_const(cPsychParser, "UTF16BE", INT2NUM(YAML_UTF16BE_ENCODING)); rb_require("psych/syntax_error"); - ePsychSyntaxError = rb_define_class_under(mPsych, "SyntaxError", rb_eSyntaxError); + ePsychSyntaxError = rb_const_get(mPsych, rb_intern("SyntaxError")); rb_define_method(cPsychParser, "parse", parse, -1); rb_define_method(cPsychParser, "mark", mark, 0); diff --git a/test/psych/test_exception.rb b/test/psych/test_exception.rb index c6d98d7a99..5615fc2db4 100644 --- a/test/psych/test_exception.rb +++ b/test/psych/test_exception.rb @@ -126,5 +126,26 @@ module Psych assert_equal 1, w.foo assert_nil w.bar end + + def test_psych_syntax_error + Tempfile.open(['parsefile', 'yml']) do |t| + t.binmode + t.write '--- `' + t.close + + begin + Psych.parse_file t.path + rescue StandardError + assert true # count assertion + ensure + return unless $! + + ancestors = $!.class.ancestors.inspect + + flunk "Psych::SyntaxError not rescued by StandardError: #{ancestors}" + end + end + end + end end |