私はそれらのコメントに対する投稿者の返答に満足している。議論になり得るような問題はないものと確信している。いかなる保証もしない(特にどこかで明示しない限り)。問題を残していないという意見の宣言です。このタグはレビューアの寄与をクレジットする働き、レビューの進捗の度合いをメンテナに知らせる働きを持ちます。タグがあなたのパッチをカーネルにマージする可能性を高めるでしょう。標準的なパッチのフォーマット標準的なパッチのサブジェクトは以下のとおりです。phrase標準的なパッチの、電子メールのボディは以下の項目を含んでいます。(変更履歴)にコピーされます。ジログ内にコピーされます。余計なコメントは、チェンジログには不適切です。実際のパッチ(差分出力)サブジェクト行のフォーマットは、アルファベット順で電子メールをとてもソートしやすいものになっています。(ほとんどの電子メールクライアントはソートをサポートしています)パッチのサブジェクトの連番は0詰めであるため、数字でのソートとアルファベットでのソートは同じ結果になります。電子メールのサブジェクト内のサブシステム表記は、パッチが適用される分野またはサブシステムを識別できるようにすべきです。phrase」はそのパッチの概要を正確に表現しなければなりません。phrase」をファイル名にしてはいけません。phrase」を使ってはいけません(「パッチシリーズ」とは順序付けられた関連のある複数のパッチ群です)。phrase」がそのパッチにとって世界で唯一の識別子になるように心がけてください。チェンジログの中へずっと伝播していきます。phrase」は、開発者が後でパッチを参照するために議論の中で利用するかもしれません。検索したがるでしょう。一目にとまる情報となるでしょう。phrase」はなぜパッチが必要であるか、パッチが何を変更するかの2つの情報をせいぜい70〜75文字で表現していなければなりません。phrase」は簡潔であり説明的である表現を目指しつつ、うまくまとめられている概要となるべきです。大括弧で閉じられたタグを接頭辞として付加してもかまいません。phrase」の一部とは考えませんが、パッチをどのように取り扱うべきかを表現します。「RFC」のようなコメントを要求するタグが挙げられます。4/4」のように番号を付けてもかまいません。これは開発者がパッチを適用する順番を確実に把握するためです。開発者がパッチシリーズの中のすべてのパッチをもらさずレビュー或いは適用するのを保証するためです。」行は電子メールのボディの一番最初の行でなければなりません。その形式は以下のとおりです。」行はチェンジログの中で、そのパッチの作成者としてクレジットされている人を特定するものです。」が、チェンジログの中でパッチの作成者を決定するために使われるでしょう。説明本体は無期限のソースのチェンジログにコミットされます。本体はそのパッチに至った議論の詳細を忘れているある程度の技量を持っている人がその詳細を思い出すことができるものでなければなりません。メッセージ等)を記載することは問題に対処可能なパッチを求めてコミットログを検索する人々にとって特に有用です。パッチがコンパイル問題を解決するのであれば、そのパッチを探している人が見つけることができる情報だけで十分であり、コンパイル時の全てのエラーを含める必要はありません。phrase」と同様に、簡潔であり説明的であることが重要です。部分を認識させるという重要な役目を果たします。追加され何行消されたかを示すものです。コマンドは特に大きなパッチにおいて役立ちます。その時点でだけ又はメンテナにとってのみ関係のあるコメントは無期限に保存されるチェンジログにとって適切ではありません。そのため、このようなコメントもマーカー行の後に書かれるべきです。バージョン間で何が変更されたかを表す「パッチの変更履歴」が挙げられます。」を指定してください(インデントを含めてちょうど80列に合うでしょう)。適切なパッチのフォーマットの詳細についてはセクション3の参考文献を参照してください。リポジトリのアドレスとブランチ名を同じ行に1行で記載してください。することで、3回の連続クリックで全て選択できます。正しい形式は下記の通りです。結果を検証して間違っていることに気づいたことがあります。どこから何を引っ張るべきかを「探したり」、正しいブランチ名かどうかを重ねてチェックしたりする必要がなくなればより快適になるでしょう)。オプションは新規ファイル、削除されたファイル、名前が変更されたファイルの概要を生成します。結果はかなり異なってきます。大規模な変更(追加と削除のペア)をファイル名の変更と判断するためです。カーネルに変更を適用することに関係のある一般的な「お約束」の多くを載せています。物事には例外というものがあります。例外を適用するには、本当に妥当な理由が不可欠です。コンピュータ・サイエンス101と呼ぶでしょう。言うまでもなく、あなたのコードがこのコーディングスタイルからあまりにも逸脱していると、レビューやコメントなしに受け取ってもらえないかもしれません。特筆すべき例外は、コードをあるファイルから別のファイルに移動するときです。この場合、コードを移動するパッチでは、移動されるコードに関して移動以外の変更を一切加えるべきではありません。これにより、コードの移動とあなたが行ったコードの修正を明確に区別できるようになります。これは実際に何が変更されたかをレビューする際の大きな助けになるとともに、ツールにコードの履歴を追跡させることも容易になります。)であなたのパッチをチェックしてください。論としてではなく、指標としてみるべきです。あなたのコードが違反はしているが修正するより良く見えるのであれば、おそらくそのままにするのがベストです。チェック:考慮する必要があるあなたはパッチに残っている全ての違反について、それがなぜ必要なのか正当な理由を示せるようにしておく必要があります。散乱したコードは、読むのもメンテナンスするのも面倒です。」関数かマクロで定義してください。後はコンパイラが、何もしない箇所を最適化して取り去ってくれるでしょう。」関数はマクロよりもずっと推奨されています。型安全性があり、長さにも制限が無く、フォーマットの制限もありません。おいては、マクロと同じくらい軽いです。」関数を使うことができないような場所(マクロの引数の文字列連結のような)にだけ使われるべきです。設計に凝りすぎるなそれが有用になるかどうか分からないような不明瞭な将来を見越した設計をしないでください。「できる限り簡単に、そして、それ以上簡単にならないような設計をしてください。