From: UENO Katsuhiro <unnie@...>
Date: 2002-01-28T20:19:09+09:00
Subject: [ruby-dev:15786] definition of StringValue

うえのです。

ruby.h を眺めてて思ったのですが、StringValue は

Index: ruby.h
===================================================================
RCS file: /src/ruby/ruby.h,v
retrieving revision 1.57
diff -u -r1.57 ruby.h
--- ruby.h	2001/12/21 07:20:23	1.57
+++ ruby.h	2002/01/28 11:11:55
@@ -201,7 +201,7 @@
 VALUE rb_str_to_str _((VALUE));
 VALUE rb_string_value _((volatile VALUE*));
 
-#define StringValue(v) if (TYPE(v) != T_STRING) rb_string_value(&(v))
+#define StringValue(v) ((TYPE(v) != T_STRING) ? (v) : rb_string_value(&(v)))
 void rb_check_safe_str _((VALUE));
 #define SafeStringValue(v) do {\
     StringValue(v);\

こうしたほうが良くないですか？


 --  ----  -     - - -- -
うえの かつひろ <unnie@blue.sky.or.jp>