Re: [MD:7279] チケット #237

["M.Fujii" <boochang@xxxxxxxxxxxx>]

藤井です。

From: Hideyuki SHIRAI (白井秀行) <shirai@xxxxxxxxxxx>
Subject: [MD:7279] チケット #237
Date: Fri, 21 Apr 2006 20:03:22 +0900 (JST)
> どなたか、チケット #237 のパッチ査読してくださいな。
> 
> http://www.meadowy.org/meadow/ticket/237
> 
> remove-framehdc-00.patch (4.6 kB) - anonymous によって04/09/06 21:45:06に追加.
>     FRAME_HDCの削除
> remove-warning-00.patch (2.1 kB) - anonymous によって04/09/06 21:45:56に追加.
>     Warningの解消パッチ
> exclusion-01.patch (8.2 kB) - anonymous によって04/16/06 16:40:25に追加.
>     排他追加(4/16版)
> 
> http://www.meadowy.org/meadow/attachment/ticket/237/remove-framehdc-00.patch?format=raw
> http://www.meadowy.org/meadow/attachment/ticket/237/remove-warning-00.patch?format=raw
> http://www.meadowy.org/meadow/attachment/ticket/237/exclusion-01.patch?format=raw
> 
> これらを当ててから、一回も落ちなくなりました。

パッチでの排他制御の妥当性については分かりませんが、非常に気になる点が
あるので意見させてください。

このパッチでは関数に引数を追加してまで frame を排他処理を行なうべきとこ
ろまで渡す処理が追加されています。このようなことまでしないと排他できな
いのなら、排他処理と DC 取得を分離し、frame を引数に取る必要なく排他で
きるように変更する必要があるのではないかと思います。

このままではいたるところで #ifdef MEADOW しなければならず、保守性に悪影
響を及ぼしそうな気がします。

--
藤井 正行 / Masayuki FUJII