[ruby-dev:25779] Re: 鬼車コードレビュー

Yukihiro Matsumoto wrote:
> st.[ch]のst_*_strendのあり方が好みでないのでソースを見ている
> のですが、以下の点に気がつきました。これがバグかどうかは私に
> は判断できませんでした。
> 
>   (1) valgrind miniruby -e '/(?<foo>.*)/' を実行すると、
> 
>       ==12825== Conditional jump or move depends on uninitialised value(s)
>       ==12825==    at 0x80AF5FA: optimize_node_left (regcomp.c:4274)
>       ==12825==    by 0x80AF968: set_optimize_info_from_tree (regcomp.c:4391)
>       ==12825==    by 0x80AFFE9: onig_compile (regcomp.c:4812)
>       ==12825==    by 0x80A32C1: make_regexp (re.c:657)
> 
>       という警告が出る。該当行は
> 
>       	    if (BIT_STATUS_AT(env->backrefed_status, en->regnum))
> 
>       なので、en->regnumが初期化されていないようだ。

バグです。
初期化されていないのは、env->backrefed_statusのほうでした。
これから修正します。

>   (2) regparse.c:name_add()で登録されたNameEntryがどこでも
>       freeされていないように思える。e->name, e->back_refsは
>       i_free_name_entry()でfreeされているようだ。
> 
> で、それはそれとしてst_*_strendはregparse.cに(renameして)移
> 動したいのですが構いませんか？

どうぞ。
(2)のバグは、そちらの作業が終わった後に
修正します。

strend関連の移動の他には、st.[ch]の変更は
ないのでしょうか？
--
小迫