From e8f43ad081e35d8779666f79f01d9c59a4122c29 Mon Sep 17 00:00:00 2001 From: matz Date: Wed, 30 Aug 2006 09:46:57 +0000 Subject: [PATCH] * object.c (rb_Integer): Integer(nil) should raise TypeError. [ruby-talk:210205] * object.c (nil_to_s): no longer returns empty string but "nil". [ruby-talk:210205] * lib/mkmf.rb: avoid COMMON_HEADERS being nil. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@10801 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 10 ++++++++++ lib/mkmf.rb | 2 +- object.c | 12 +++++++----- 3 files changed, 18 insertions(+), 6 deletions(-) diff --git a/ChangeLog b/ChangeLog index 9addf10942..67db2f0715 100644 --- a/ChangeLog +++ b/ChangeLog @@ -65,6 +65,16 @@ Fri Aug 25 16:05:50 2006 Nobuyoshi Nakada * object.c (sym_call): check if the receiver is given. +Fri Aug 25 01:10:11 2006 Yukihiro Matsumoto + + * object.c (rb_Integer): Integer(nil) should raise TypeError. + [ruby-talk:210205] + + * object.c (nil_to_s): no longer returns empty string but "nil". + [ruby-talk:210205] + + * lib/mkmf.rb: avoid COMMON_HEADERS being nil. + Wed Aug 23 00:25:14 2006 Nobuyoshi Nakada * lib/rexml/source.rb (REXML::IOSource#initialize): encoding have to diff --git a/lib/mkmf.rb b/lib/mkmf.rb index b019b5f755..c5d9b194e3 100644 --- a/lib/mkmf.rb +++ b/lib/mkmf.rb @@ -1501,7 +1501,7 @@ end config_string('COMMON_HEADERS') do |s| Shellwords.shellwords(s).each {|s| hdr << "#include <#{s}>"} end -COMMON_HEADERS = (hdr.join("\n") unless hdr.empty?) +COMMON_HEADERS = if hdr.empty? then "" else hdr.join("\n") end COMMON_LIBS = config_string('COMMON_LIBS', &split) || [] COMPILE_RULES = config_string('COMPILE_RULES', &split) || %w[.%s.%s:] diff --git a/object.c b/object.c index c8c39a3695..a396da0913 100644 --- a/object.c +++ b/object.c @@ -676,17 +676,15 @@ nil_to_f(VALUE obj) /* * call-seq: - * nil.to_s => "" + * nil.to_s => "nil" * - * Always returns the empty string. - * - * nil.to_s #=> "" + * Always returns the string "nil". */ static VALUE nil_to_s(VALUE obj) { - return rb_str_new2(""); + return rb_str_new2("nil"); } /* @@ -2005,6 +2003,10 @@ rb_Integer(VALUE val) case T_STRING: return rb_str_to_inum(val, 0, Qtrue); + case T_NIL: + rb_raise(rb_eTypeError, "can't convert nil into Integer"); + break; + default: break; }