From: 青柳政聡 <mas...@gm...> - 2011-02-04 11:12:02
|
青柳です。 >> そこで、その前にソースのスタイルがK&RからANSIに変更になったが統一されていないようなので、 >> 以下のような方針でスタイルの統一をすることを提案します。 >> * スタイルの変更はスクリプトを書いて、機械的にやってしまう >> * 確認方法 >> * 差分のコードレビュー >> * 変更前後でバイナリ(soファイル)が同一になることを確認 >> (コンパイルオプションから"-g"を削除して確認) >> #ifディレクティブで切り替えている箇所は、バイナリの確認から漏れますが、コードレビューと >> バグ報告に期待ということで、大目に見ます。 > > いいと思います。 > 「差分のコードレビュー」は1ファイルずつコミットしてもらえれ > ばなんとかなるかなぁという気がします。 1ファイルずつコミットする必要があるのでしょうか? パッケージ単位ぐらいでコミットして、変更されたファイルのリストがあれば いいかなと思っていたのですが、いまいち意図がよく分からないです。 それと、引数の宣言漏れがいくつ見つかりましたので、その修正をコミットしたいと思います。 差分を添付いたしますので、ご確認ください。 こちらではビルドが通るところまでは確認しています。 なお、gdkdisplay_get_double_click_distanceについては、引数自体が必要ないと思われたので、 引数の方を削っています。 |