[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[FDclone-users:00897] Re: MINIX3 patch



稲員です。

On Sun, 04 Jul 2010 04:28:01 +0900
Takashi SHIRAI <shirai@unixusers.net> wrote:

>  一点だけ懸念点を。uid_t/gid_t を int で受けている箇所が幾
> つかありますが、(uid_t)-1 と (int)-1 は異なる値なのでそれを
> 同一視して変換しないといけません。
>  compiler の暗黙型変換やキャストでは賄い切れないので、ちゃ
> んと変換するのは結構手間が掛かります。


ああ、一応気にはなってて WRITE_DIR 対応と共に検討事項に入れてはいたん
ですけど、やはり。


>  この点は ACK の古典モダン混在に関するバグなので、uid_t 絡
> みの関数だけモダンタイプに書換えてしまった方が簡単に実装出来
> ると思います。


確かにそれが一番素直ですね。迂遠なことをしていた上に BUG 付きでした。
その方針で書き換えたものを付けておきます。 fd300h.minix3.v2.patch.gz を
当てた状態からの差分です。追加で設けていた CCSHORTARG, SHORTUID などの
識別子も削除して __ACK__ で分けるようにしました。

その他にも先の patch には問題があったのでその修正も兼ねています。
どうやら余計な変更をした revision から差分を取ってしまったようです。


-- 
Rikito INAKAZU (稲員力士) <riki1017kazu@gmail.com>

Attachment: fd300h.minix3.v2a.patch.gz
Description: Binary data