[ruby-dev:35695] Re: PENDINGS.rb (Was: Re: [Ruby 1.9 - Bug #354] (Assigned) Test failure test/ruby/test_transcode.rb)

遠藤です。

2008/07/30 7:33 Tanaka Akira <[e-mail]>:
> In article <[e-mail]>,
>  "Yusuke ENDOH" <[e-mail]> writes:
>
>> 2008/07/25 2:41 NARUSE, Yui <[e-mail]>:
>>> 行単位の restart は ISO-2022-JP や UTF-7 は可能な気がします。
>>> つまるところ、decode しなくても改行の認識が可能な encoding については、
>>> gets をサポートする方向です。
>>
>> test_terminator_stateful_conversion(TestIO_M17N) のように、gets に
>> デリミタを指定するようなのも、サポート対象外ですよね。たぶん。
>
> なんでサポート対象外なんですか?

改行の認識の話ばっかりだったので、そういう前提かと誤読してました。


ちょっと UTF-7 を調べてみたんですが、RFC 2152 によると任意の文字を
BASE64 でエンコードしてもいいようです。

$ echo foo+AAo-bar | iconv -f=UTF-7 -t=UTF-8 | od -c
0000000   f   o   o  \n   b   a   r  \n
0000010

これでは改行文字が認識できない可能性があるので、UTF-7 に対する
gets は一切禁止しなければならないと思います。
それならいっそ dummy encoding に対する gets はすべて禁止しちゃえ、
というのが私の意見です。

が、どうしても ISO-2022-JP で gets したいという需要が無視できない
のでしたら、gets で ISO-2022-JP だけ特別扱いするのはどうでしょう。
IO が stateful encoding をサポートする日までの workaround です。

ESC 以外の制御文字だけからなるデリミタだけは通すようにしました。

Index: io.c
===================================================================
--- io.c	(revision 18264)
+++ io.c	(working copy)
@@ -1956,8 +1956,23 @@

     GetOpenFile(io, fptr);
     rb_io_check_readable(fptr);
-    if (rb_enc_dummy_p(io_input_encoding(fptr)) && rs != rb_default_rs) {
-	rb_raise(rb_eNotImpError, "gets with delimiter against dummy
encoding is not currently supported");
+    /* FIXME! please support stateful encoding */
+    if (rb_enc_dummy_p(io_input_encoding(fptr))) {
+	const char *name = rb_enc_name(io_input_encoding(fptr));
+	if ((strcmp(name, "ISO-2022-JP")  == 0 || strcmp(name, "ISO2022-JP")  == 0 ||
+	     strcmp(name, "ISO-2022-JP2") == 0 || strcmp(name, "ISO2022-JP2") == 0) &&
+	    !NIL_P(rs) && rs != rb_default_rs) {
+	    int i;
+	    for (i = 0; i < RSTRING_LEN(rs); i++) {
+		char c = RSTRING_PTR(rs)[i];
+		if (c == 0x1b || 0x20 <= c) {
+		    rb_raise(rb_eNotImpError, "gets against dummy encoding is not
currently supported");
+		}
+	    }
+	}
+	else {
+	    rb_raise(rb_eNotImpError, "gets against dummy encoding is not
currently supported");
+	}
     }
     if (NIL_P(rs)) {
 	str = read_all(fptr, 0, Qnil);

-- 
Yusuke ENDOH <[e-mail]>