今日はリファクタリングが少しだけでした。
nobu: r47257 2014-08-23 11:52:23 +0900
ruby.c の process_options() で RARRAY_ASET() の引数に関数呼び出しなどを渡していたため、マクロ展開時に RARRAY_PTR_USE_START() から RARRAY_PTR_USE_END() の間に処理が沢山入ってしまうので、あらかじめ引数に渡す値を変数に入れておくようにしています。現状の RARRAY_PTR_USE() の処理をみて、特にここで処理が多いと困る点は見受けられませんでしたが、一般的にマクロの引数は複数回評価されたりする可能性を考慮して関数呼び出しなどの式は避けるほうがいいのでそういうことかもしれません。
svn: r47258 2014-08-23 11:52:34 +0900
version.h の日付更新。
nobu: r47259 2014-08-23 22:32:32 +0900
win32/win32.c の cmdglob() でバッファ内の文字を変換する処理を translate_char() という既に存在していた inline 関数を利用するようにリファクタリングしています。