From: "NARUSE, Yui" <naruse@...>
Date: 2008-01-26T10:04:32+09:00
Subject: [ruby-dev:33394] Re: [ruby-cvs:22481] Ruby:r15244 (trunk): * string.c (rb_str_usascii_new{,2}: defined.

成瀬です。

U.Nakamura wrote:
> こんにちは、なかむら(う)です。
> 
> In message "[ruby-dev:33380] Re: [ruby-cvs:22481] Ruby:r15244 (trunk): * string.c (rb_str_usascii_new{,2}: defined."
>     on Jan.26,2008 02:07:43, <akr@fsij.org> wrote:
> | r15241 で test-all を動くようにしたのですが、それもつかのま
> | 
> | In article <200801251640.m0PGe4K3016900@ci.ruby-lang.org>,
> |   naruse@ruby-lang.org writes:
> | 
> | > naruse	2008-01-26 01:40:02 +0900 (Sat, 26 Jan 2008)
> | >
> | >   New Revision: 15244
> | 
> | によって動かなくなったようです。
> 
> ちょっと見てみて、これ自体は簡単に直せることはわかったんです
> が、rb_str_new系でptrを0で呼んでるところは全部チェックして問
> 題がないか確認する必要がありそうです。
> つまり、ptrが0の時にcoderangeを設定してしまうのは相当危険なん
> じゃないかと思いますが、どうしましょうか。

なるほど、発想としては ChangeLog の通り、空文字の場合は設定してしまうと
いうものだったのですが、根本的に副作用が大きいのですかねぇ。

とりあえず len == 0 の場合に変えてみました。

-- 
NARUSE, Yui  <naruse@airemix.com>
DBDB A476 FDBD 9450 02CD 0EFC BCE3 C388 472E C1EA