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

こんにちは。山本です。

>> ２．Next(p)をより安全にして、必ず＋１以上するようにした。
>
>'\0'を指しているときにも次を見てしまうことのほうがおかしいよう
>な気がしますが、具体的にはどこで発生するんでしょうか。

初めは

  #define Next(p) ((p) + 1) /* single byte environment */
  #define Next(p) (CharNext(p)) /* multi byte environment */

としていましたが、これだと mbyte-env でビルドしたとき、
"comiling WIN32API..." から進まなくなりました。sbyte-env ではビルドできるので、
マクロの挙動に違いがあるのではないかと思い、調べました。すると、
'\0' を指しているときの Next() の挙動に違いがあったので、

  #define Next(p) (*(p) ? CharNext(p) : (p) + 1) /* multi byte environment */

と変更したところ、コンパイルできました。このあたりの顛末は [ruby-dev:22089] を参考にしてください。

・・・・・・・・・・・・

次に、mblen がエラーのとき -1 を返すらしいので、それはまずいと思い
必ず +1 以上進むようにしました。そのとき、念のため CharNext にも同様の変更をしました。

ただ、関数呼び出しの分遅くなるし、Siena.さんによると（[ruby-dev:22115]）
他のモジュールでも mblen のエラーはチェックしてないそうなので、この変更はやめたほうがいいかもしれません。

>> +	int ret = ::CompareString(LOCALE_SYSTEM_DEFAULT,
>
>これはCでは通らないと思います。

あ、そうですね。Cに名前空間はありませんでした。

># あとは、インデントがバラバラだとか、CamelCaseとそうでないのが
># 混じってるのはヤだなぁ、とか。

うーん、他の部分のインデントに合わせたつもりだったのですが・・・。普段は

for (int ...)
	for (int ...)
		if

みたいに全部インデントするんですが、それだと浮きまくりだったので、適当に縮めました。

CamelCase は半ばわざとです。まだ動作が問題ないかわからないし、
Next や Inc が目立ったほうがデバッグしやすいので・・・
動作が問題なければ、パフォーマンスを改善して、それから書式を整えたいです。