[ruby-dev:22163] Re: Dir.glob と Shjift_JIS について

>> >全然オーバーヘッドを気にしているわけではなくて、ターミネータを
>> >越えてアクセスしようとするところが、バッファオーバーランを誘発
>> >しそうでなんとなく気になってます。
>> 
>> でも、(p) + 1 を使うシングルバイト環境では、p == '\0' でも１増えますよね。
>> Next() の動作が環境によって異なるのは避けたいです。
>> mblen()はターミネータを越える数を返さないだろうし・・・
>
>私見ですが，速さと安全性を天秤にかけるなら速さを犠牲にしてでも安全側に
>倒すべきだとおもいます。

うーん・・・・
(greater((p) + 1, (p) + mblen(p, INT_MAX))) って危険ですか？
仮に p がターミネータを指しているときにポインタを進めないようにしても、
mblen()がおかしな値を返せば危険なわけで、それを防ぐには
返された値をもとに、ターミネータを超えてないか for ループで調べなくてはなりません。

#define Next(p) NextImpl(&(p))

static char *
NextImpl(p)
    char *p
{
    if (!*p)
        return p;

    int n = mblen(p, INT_MAX);

    if (n <= 0) {
        return p + 1; // raise?
    }

    for (n--) {
        ++p;
        if (!*p)
            break;
    }

    return p;
}

みたいに。ここまでやる必要があるでしょうか？

mblen()が仕様どおりに組まれていれば、p が '\0' を指しているときは、mblen() は 0 を返すので、
+2 でも +3 でもなく、+1 になるので、危険はないと思います。
シングルバイト環境は別に (*(p) ? (p) + 1 : (p)) なんてやってないですし。

Next()の動作が環境依存なほうが、原因不明のバグにはまりそうで怖いです。（現にはまった）