diff options
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | error.c | 6 | ||||
-rw-r--r-- | test/ruby/test_iseq.rb | 3 |
3 files changed, 12 insertions, 3 deletions
@@ -1,3 +1,9 @@ +Fri Aug 12 12:50:31 2016 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * error.c (rb_syntax_error_append): fix newline in syntax error + message to the beginning, not after file name and line number. + [Feature #11951] + Thu Aug 11 16:24:23 2016 nerdinand <nerdinand@nerdinand.com> * compar.c (cmp_clamp): Introduce Comparable#clamp. [Feature #10594] @@ -106,16 +106,16 @@ rb_syntax_error_append(VALUE exc, VALUE file, int line, int column, } else { VALUE mesg; - const char *pre = NULL; if (NIL_P(exc)) { mesg = rb_enc_str_new(0, 0, enc); exc = rb_class_new_instance(1, &mesg, rb_eSyntaxError); } else { mesg = rb_attr_get(exc, idMesg); - pre = "\n"; + if (RSTRING_LEN(mesg) > 0 && *(RSTRING_END(mesg)-1) != '\n') + rb_str_cat_cstr(mesg, "\n"); } - err_vcatf(mesg, pre, fn, line, fmt, args); + err_vcatf(mesg, NULL, fn, line, fmt, args); } return exc; diff --git a/test/ruby/test_iseq.rb b/test/ruby/test_iseq.rb index dba714aa62..cdfa380edd 100644 --- a/test/ruby/test_iseq.rb +++ b/test/ruby/test_iseq.rb @@ -234,6 +234,9 @@ class TestISeq < Test::Unit::TestCase end end assert_equal([m1, e1.message], [m2, e2.message], feature11951) + e1, e2 = e1.message.lines + assert_send([e1, :start_with?, __FILE__]) + assert_send([e2, :start_with?, __FILE__]) end def test_translate_by_object |