|
．X の locale - あれは当初から XLC_LOCALE に並べる順番で決まっていたような。EUC-JP-MS もそうだけど、iconv_open() のレベルで変換表のバリエーションを選択するしくみが何か欲しいところではありますな。
．lib/36938 については、読み返してみたら、そもそも論点は EILSEQ だけなんですね。mbrtowc_priv が -2 を返しているような場合はもともとちゃんと扱っている。だからそれと同様に
state = *psenc;
err = _FUNCNAME(mbrtowc_priv)(ei, pwc, (const char **)&s, n, psenc, &nr);
if (err) {
+ /* In error case, we should restore the state. */
+ *psenc = state;
*nresult = -1;
return (err);
}
if (nr == (size_t)-2) {
*psenc = state;
*nresult = -1;
return (err);
}
でいいんじゃ。もっと整理すれば、
state = *psenc;
err = _FUNCNAME(mbrtowc_priv)(ei, pwc, (const char **)&s, n, psenc, &nr);
+ if (nr == (size_t)-2)
+ err = EILSEQ;
if (err) {
+ /* In error case, we should restore the state. */
+ *psenc = state;
*nresult = -1;
return (err);
}
- if (nr == (size_t)-2) {
- *psenc = state;
- *nresult = -1;
- return (EILSEQ);
- }
．なお、毎回 chlen を 0 にする必要はないのです。stateless enc なら、上の二つのケースを除けば自動的に initial state に戻りますから。
．いっそ _ENCODING_IS_STATE_DEPENDENT マクロで「stateless enc なら常にスタック上のスクラッチな state 変数を使う」と書き換えてしまってもいいかも（御先祖様の「壊れてない物は直すな」という声が聞こえてきたが）。もともとこのファイルは evil な存在です。
．エラーというものに対する私の先入観が原因だよなこれ。-2 のケースはわざわざ元に戻してるんだもん。そうじゃないエラーはハナっから復帰不可能と決めつけているコードだ。実際のところ、規格書のどこにもエラーの時に副作用を発生させて良いとは書いてないから、やっぱり戻さないとマズい気がする。
．しかしこうなると、他の関数もエラーの時には状態を戻さないといけないような。
|
