[Message Prev][Message Next][Thread Prev][Thread Next][Message Index][Thread Index]

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



藤井です。

From: yamazaki-k@xxxxxxxxxxxxxxxxx
Subject: Re: [MD:7279] チケット #237
Date: Sat, 22 Apr 2006 22:46:33 +0900 (JST)
> また、藤井さんのご指摘の関数の引数追加ですが、exclusion-00.patchでは、
> 引数を追加していましたが、白井さんのコメントでVC6ではコンパイル
> できなかったとのことで、再確認したところ、上位で排他がかかっていたため、
> exclusion-01.patchでは、この修正を取り消しています。

すみません。exclusion-01.patch は exclusion-00.patch に対する追加ではな
くて修正なのですね。大ハズレなコメントをして申し訳ありません。

> > パッチでの排他制御の妥当性については分かりませんが、非常に気になる点が
> > あるので意見させてください。
> > 
> > このパッチでは関数に引数を追加してまで frame を排他処理を行なうべきとこ
> > ろまで渡す処理が追加されています。このようなことまでしないと排他できな
> > いのなら、排他処理と DC 取得を分離し、frame を引数に取る必要なく排他で
> > きるように変更する必要があるのではないかと思います。
> > 
> > このままではいたるところで #ifdef MEADOW しなければならず、保守性に悪影
> > 響を及ぼしそうな気がします。

ということで、保守性が云々という意見は取り下げさせてください。すみません。

--
藤井 正行 / Masayuki FUJII