[ruby-dev:38443] Re: [ANN] Ruby 1.9.1-p129リリース

こんにちは。

Ruby 1.9.1-p129のリリース、おめでとうございます。

いくつか手元でコンパイル時のウォーニングで気がついたことを報告します。

1. ext/etc/etc.c

	uid_tが再定義されたと出ます。

#ifndef HAVE_TYPE_UID_T
#define uid_t int
#endif

	となっていて、uid_tが存在することはconfigureで確認できています
	が、ここ以外にHAVE_TYPE_UID_Tは使用されている箇所を見つけられ
	ませんでした。

	コンパイルしていたNetBSD 5.99.11ではuid_tはuint32_tで、符合の
	有無の異なるタイプ間で再定義されるのは、あまり気持ちが良いもの
	ではありません。(この場合の実害はあまりないでしょうけど。)

2. ext/pty/pty.c

	getDevice()は戻り値がvoid な関数にも関わらず、値を返しています。


--- ext/pty/pty.c.orig	2008-12-26 01:33:06.000000000 +0900
+++ ext/pty/pty.c
@@ -422,7 +422,7 @@ getDevice(int *master, int *slave, char 
 	rb_gc();
 	get_device_once(master, slave, SlaveName, 1);
     }
-    return Qnil;
+    return;
 }
 
 /* ruby function: getpty */


3. ext/stringio/stringio.c

	strio_ungetbyte()に無効なコードがあります。
	消し忘れでしょうか?

--- ext/stringio/stringio.c.orig	2009-02-22 21:49:06.000000000 +0900
+++ ext/stringio/stringio.c
@@ -744,7 +744,6 @@ strio_ungetc(VALUE self, VALUE c)
 static VALUE
 strio_ungetbyte(VALUE self, VALUE c)
 {
-    NUM2INT(c);
     return strio_ungetc(self, c);
 }
 

以上です。

-- 
神戸 隆博(かんべ たかひろ)		at 仕事場