etc.2010-02-06コードのレビューをもらうためにはこのエントリの続き。昨日良い例見つけたから、それを例にして書く。OSS関係なくて、ソフトウェアの開発でよくあること。この例で出てきてる方、例に使っちゃってすみません。APIがクソなんだから機能足そうよってだけなんだよね、簡単に言えば。Saitoさんは、ParseEnumValueを直接呼ばなくて文字列を組み立て直してから、ParseEnumValueを呼ぶようにしてる。ここで文字列組み立て直す必要あんの?もっとEasyにやろうよって思うんだよね。(ついでにそういうシンプルなパッチ書いたけど)。文字列を組み直すところのやり方は非常にダメ。APIが非常に用意されているので、それを使えばいいのに、ANSI文字列を作ってそれをUNICODEに変換なんて発想がひどすぎ。すくなくとも直接UNICODEの文字列作ればいいのにさ。ちなみに、Mozilla的な感じだと、nsString::AppendIntとかnsString::Assignを駆使するのがベターで、ちょー簡単。どのOSでもアプリでもOSSじゃなくてもプロプラでもそうだけど、大人数で作るプログラムというのは、豊富なライブラリも用意されてて、それを使うようにコーディングルールが作られている。(そうじゃなければチェックインできないよ)。日本人にとってはBorisの言い方はキツいのかもしれないけど、まだちゃんと細かく書いてくれるだけ、Boris優しいって思うよ。彼の言うように直せばたぶんSaitoさんの貢献がバグ修正という形になったと思うんだけど、そこでイヤになったか忙しくなったか、別の理由とかでそれ以上はパッチを書かなかったってのが、現在の状態ってところ。彼は別のところを直そうともしてるんだけど、このバグで一緒にやるから面倒なことになる。それをやるんだったら、新しいバグ立てるべきなんだ。(そういう会社に居たときに、こういうこと分からない人が結構居てて、そういう人には説教してた)。アメリカ人というか欧米人だったらそうやるのが普通だから、それに従ってやるべきなんだ。WebKitにも存在してたこと。