[ruby-dev:23189] Re: File.fnmatchのリファクタリング

山本です。

>なかだです。
>
>At Fri, 12 Mar 2004 14:42:34 +0900,
>H.Yamamoto wrote in [ruby-dev:23187]:
>> これは予定通り行うつもりです。問題があれば報告お願いします。
>
>問題というわけではないのですが一つだけ感想です。
>
>> +static char *
>> +bracket(p, s, flags)
>> +    const char *p; /* pattern (next to '[') */
>> +    const char *s; /* string */
>
>わざわざこういうコメントをつけるくらいなら、変数名自体を変えて
>はどうでしょうか。

頻繁に使う変数だし、ptmp などの変数との兼ね合いもあるので、この方が読みやすいと思います。

    if (pat_tmp && str_tmp) {
	pat = pat_tmp;
	Inc(str_tmp); /* !ISEND(*str_tmp) */
	str = str_tmp;
	continue;
    }

よりもこちらの方が好きです。