From: 青柳政聡 <mas...@gm...> - 2011-02-06 08:32:43
|
青柳です。 K&RからANSIへのスタイル変更の報告です。 バイナリが一致しなくなってしまうことがあるようで、現在のところバイナリ一致が 確認出来た範囲をコミットしてあります。 バイナリ一致しない原因は、引数をいきなりキャストしているようなケースで、 最適化がされているのではないかという印象を持っています。 ただ、"-O0"指定をしても関係ないようでした。 > どのファイルが変更されたかどうかを知りたいわけではなく、大き > なdiffを1つ見るよりも、小さなdiffを複数見る方が私は見やすいの > です。 > > 1ファイルずつコミットするのが大変というのであれば、まとめて > でもかまいません。ただ、私以外にdiffを見ている人がいるかどう > かわからないので、「差分のコードレビュー」の効果はあまり期待 > しないでください。。。 一応ファイル毎にコミットしてありますが、かなり数が多いので須藤さんの方では さらっと見る程度でいい気もします。 (こちらでコードレビューも、割としっかりやったつもりなので) 残っているバイナリが一致しなくなってしまう変更箇所を、コードレビューのみで コミットしてしまいたい気もするのですが、いかがでしょうか? 差分を添付いたします。 |