アカウント名:
パスワード:
問題のパッチは、56ファイル13万行で4.6MBありますが、日立のファイバチャネルボード用のドライバ(hfcldd)を新規追加するもののようです。基本的に「複数のパッチに分割」なんてできそうにないと思いますが、いったいどうしろというのでしょうか?…
対応機種が多いのか、hfcl_detect.c と hfcl_detect_fx.c で3万行取られてるのが大きい感じですので、「対応機種の少ないhfclddドライバを新規追加」と「hfclddに対して対応機種を増やすパッチ」という形なら分割できそうですが…無駄に手間をかけるだけだよなぁ。
Johannesが提言したように「論理的に」分割しましょう。逆に、あなたがレビューする立場であると考えてみてください。ひとつのファイルだからといって、13万行のソースを一気読みしますか?普通は、まず大まかな流れを掴み、その後に個々のディテールを読み下す、というような流れになると思います。つまり、このレビューを助けるような形にソースコードをパッチとして分割するわけです。
パッチを分割するのは、メールが大きくなりすぎるからというわけではなく、読む「人」が理解しやすいようにする、というのが本来の理由なのです。
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
皆さんもソースを読むときに、行と行の間を読むような気持ちで見てほしい -- あるハッカー
「論理的に複数のパッチに分割」できない時はどうするの? (スコア:3, 参考になる)
問題のパッチは、56ファイル13万行で4.6MBありますが、
日立のファイバチャネルボード用のドライバ(hfcldd)を新規追加するもののようです。
基本的に「複数のパッチに分割」なんてできそうにないと思いますが、いったいどうしろというのでしょうか?…
対応機種が多いのか、hfcl_detect.c と hfcl_detect_fx.c で3万行取られてるのが大きい感じですので、
「対応機種の少ないhfclddドライバを新規追加」と「hfclddに対して対応機種を増やすパッチ」という形なら分割できそうですが…無駄に手間をかけるだけだよなぁ。
Re:「論理的に複数のパッチに分割」できない時はどうするの? (スコア:1)
Johannesが提言したように「論理的に」分割しましょう。
逆に、あなたがレビューする立場であると考えてみてください。ひとつのファイルだからといって、13万行のソースを一気読みしますか?
普通は、まず大まかな流れを掴み、その後に個々のディテールを読み下す、というような流れになると思います。
つまり、このレビューを助けるような形にソースコードをパッチとして分割するわけです。
パッチを分割するのは、メールが大きくなりすぎるからというわけではなく、読む「人」が理解しやすいようにする、というのが本来の理由なのです。