diff options
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | iseq.c | 5 | ||||
-rw-r--r-- | test/lib/test/unit/assertions.rb | 12 | ||||
-rw-r--r-- | test/ruby/test_iseq.rb | 18 | ||||
-rw-r--r-- | version.h | 6 |
5 files changed, 40 insertions, 6 deletions
@@ -1,3 +1,8 @@ +Sat Nov 19 11:48:47 2016 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * iseq.c (iseqw_s_compile_file): deal with syntax error as well as + compile, and should not abort when rescued. + Wed Nov 16 23:40:29 2016 CHIKANAGA Tomoyuki <nagachika@ruby-lang.org> * vm_eval.c (vm_call0_body): refined module should not be skipped as @@ -812,8 +812,7 @@ static VALUE iseqw_s_compile_file(int argc, VALUE *argv, VALUE self) { VALUE file, line = INT2FIX(1), opt = Qnil; - VALUE parser; - VALUE f; + VALUE parser, f, exc = Qnil; NODE *node; const char *fname; rb_compile_option_t option; @@ -827,8 +826,10 @@ iseqw_s_compile_file(int argc, VALUE *argv, VALUE self) parser = rb_parser_new(); node = rb_parser_compile_file(parser, fname, f, NUM2INT(line)); + if (!node) exc = GET_THREAD()->errinfo; rb_io_close(f); + if (!node) rb_exc_raise(exc); make_compile_option(&option, opt); diff --git a/test/lib/test/unit/assertions.rb b/test/lib/test/unit/assertions.rb index 9a1cacd5a9..975d343ba4 100644 --- a/test/lib/test/unit/assertions.rb +++ b/test/lib/test/unit/assertions.rb @@ -539,7 +539,8 @@ EOT faildesc end - def assert_in_out_err(args, test_stdin = "", test_stdout = [], test_stderr = [], message = nil, **opt) + def assert_in_out_err(args, test_stdin = "", test_stdout = [], test_stderr = [], message = nil, + success: nil, **opt) stdout, stderr, status = EnvUtil.invoke_ruby(args, test_stdin, true, true, **opt) if signo = status.termsig EnvUtil.diagnostic_reports(Signal.signame(signo), EnvUtil.rubybin, status.pid, Time.now) @@ -561,6 +562,15 @@ EOT end end end + unless success.nil? + a.for("success?") do + if success + assert_predicate(status, :success?) + else + assert_not_predicate(status, :success?) + end + end + end end status end diff --git a/test/ruby/test_iseq.rb b/test/ruby/test_iseq.rb index 4561eeb952..9f9f5418e0 100644 --- a/test/ruby/test_iseq.rb +++ b/test/ruby/test_iseq.rb @@ -1,4 +1,5 @@ require 'test/unit' +require 'tempfile' class TestISeq < Test::Unit::TestCase ISeq = RubyVM::InstructionSequence @@ -212,4 +213,21 @@ class TestISeq < Test::Unit::TestCase at_exit { assert_equal([:n, :x], Segfault.new.segfault.sort) } end; end + + def test_compile_file_error + Tempfile.create(%w"test_iseq .rb") do |f| + f.puts "end" + f.close + path = f.path + assert_in_out_err(%W[- #{path}], "#{<<-"begin;"}\n#{<<-"end;"}", /compile error/, /keyword_end/, success: true) + begin; + path = ARGV[0] + begin + RubyVM::InstructionSequence.compile_file(path) + rescue SyntaxError => e + puts e.message + end + end; + end + end end @@ -1,10 +1,10 @@ #define RUBY_VERSION "2.3.3" -#define RUBY_RELEASE_DATE "2016-11-16" -#define RUBY_PATCHLEVEL 219 +#define RUBY_RELEASE_DATE "2016-11-19" +#define RUBY_PATCHLEVEL 220 #define RUBY_RELEASE_YEAR 2016 #define RUBY_RELEASE_MONTH 11 -#define RUBY_RELEASE_DAY 16 +#define RUBY_RELEASE_DAY 19 #include "ruby/version.h" |