[ruby-dev:33925] Re: UTF_16LE.dummy? returns false

成瀬です。

NARUSE, Yui wrote:
> U.Nakamura wrote:
>> | > Encoding.compatible?(Encoding::US_ASCII, Encoding::US_ASCII)
>> | => #<Encoding:ASCII-8BIT>
>> | 
>> | Encoding.compatible? が encoding オブジェクトも引数に取るべきという主張
>> | はありかもしれません。
>>
>> encodingオブジェクトは特例として通して欲しい気はしますね。
>> 現状だと、encodingオブジェクトが別のencodingオブジェクトと
>> compatibleかどうかを知るためにはいちいち文字列作らないといけ
>> ないということですよね。
> 
> encoding から引く場合ですと、うかつにやると空文字の例外にひっかかって
>> Encoding.compatible?("".force_encoding("iso-2022-jp"),
>>  "".force_encoding("UTF-16BE"))
>> => #<Encoding:ISO-2022-JP (dummy)>
> なんてことになるので、Encoding.compatible? が欲しいかもしれません。
> 
> もっとも、結局やってくれることはと言えば、
> if enc1 == enc2 then enc1
> elsif enc1 == US-ASCII && enc2.ascii_compat? then enc2
> elsif enc2 == US-ASCII && enc1.ascii_compat? then enc1
> else nil
> end
> くらいではあるのですが。

以上のようなことをやるようにするパッチです。
--- encoding.c  (revision 15591)
+++ encoding.c  (working copy)
@@ -661,8 +661,30 @@ rb_enc_compatible(VALUE str1, VALUE str2
     idx1 = rb_enc_get_index(str1);
     idx2 = rb_enc_get_index(str2);

-    if (idx1 < 0 || idx2 < 0)
-        return 0;
+    if (idx1 < 0 || idx2 < 0) {
+       int idx1 = rb_to_encoding_index(str1);
+       int idx2 = rb_to_encoding_index(str1);
+       if (idx1 < 0 || idx2 < 0) {
+           return 0;
+       }
+       else {
+           /* str1 and str2 are encoding object */
+           enc1 = rb_enc_from_index(idx1);
+           enc2 = rb_enc_from_index(idx2);
+           if (idx1 == idx2 ) {
+               return rb_enc_from_index(idx1);
+           }
+           else if (idx1 == rb_usascii_encindex() &&
rb_enc_asciicompat(enc2)) {
+               return enc2;
+           }
+           else if (idx2 == rb_usascii_encindex() &&
rb_enc_asciicompat(enc1)) {
+               return enc1;
+           }
+           else {
+               return 0;
+           }
+       }
+    }

     if (idx1 == idx2) {
        return rb_enc_from_index(idx1);


-- 
NARUSE, Yui  <[e-mail]>
DBDB A476 FDBD 9450 02CD 0EFC BCE3 C388 472E C1EA