タグ

CodeReviewとWebKitに関するraimon49のブックマーク (2)

  • 瞬くあいだに時はながれて - steps to phantasien

    最初のバグ修正 をレビューしてくれたのはたぶん、Apple の Darin Adler だったと思う。雑誌連載のねたづくりに書いたパッチだった。このあとも Darin Adler には度々レビューをしてもらった。私にとって、Bugzilla 界隈でのメンターはこのひとだ。あるとき ASSERT_NO_EXCEPTION という小粋なマクロを私が発明した際も、彼は私にかわりメーリングリストで宣伝をしてくれた。何かをわかりあえたと思った。 最初のリグレッションとその修正は 6 つ目と 7 つ目のパッチで…仕事にする前からエンバグしてたわけですが… KDE の Nicholas Zimmerman が見てくれた。彼はそのあと BlackBerry の会社に入ったと聞いたけど、最近は姿をみない。当時の SVG は無法地帯で、やんちゃなパッチもけっこう見逃してもらえた。Zimmerman, KSVG

  • 下から目線のコードレビュー - steps to phantasien

    WEB+DB の新しいやつがちょっと前にでてます. コードレビュー特集だそうな. 時が経つのは早い. まだ次の原稿書いてないのに… そういえば前にコードレビューの話を書いた気がして, 見なおしたところ かきかけ だった. せっかくなので続きを書いてみることにします. といっても何書くつもりだったか覚えてないのでだらだらと. WEB+DB PRESS の特集は, 主にこれからコードレビューを導入したい人に向けて書かれている. 幸か不幸か私はコードレビューを義務付けれたプロジェクトで働いているため, 導入には苦労していない. かわりにレビューをちょろまかせない面倒はある. ある意味でコードレビューを <やらされている>. もちろんこの言い分は大げさだ. 必要性に異議を唱える気はない. ただ異議はさておき自分の意向とは無関係にコードレビューに参加している気分を書いた話は あまり目にしないので,

    raimon49
    raimon49 2012/12/30
    指摘される側の気分良し悪しと、リリースまでにコードレビュー必須な場合のちょっとしたテクニック集。ハンコ依頼のrs(Rubber stamp)とTBR(To Be Reviewed)の話が面白い。
  • 1