あまりにもショッキングだったソースコード | Liberty Sword

あまりにもショッキングだったソースコード

ここいらでおまいらにあまりにもショッキングなソースコードを見せてやろうか。


void foo() {
 略
}


int main() {
 略
 if (foo() == false) {
  エラー処理
 }
 略
}


雰囲気はこんな感じ。

わっかるかなー。
わっかるよねー。普通。


foo()の返り値を返さないなのに、比較を行っているのですなー。
普通、コンパイル時のWarningとかでわかりますよねぇ。
でも実際、リリース直前の際の際まで気づかれなかったわけですなー。
おかげで奇妙な不具合が発生して、そりゃもう大変大変。
こんな単純なミスを特定するために、多くの時間と人手が要されました。

はじめて見たとき、ひっくり返りそうになりましたよ。
たのむよぉ、おぉい。 (ノД`)