「C言語の switch文のdefault抜け」記事へのコメント

Callerを調べたほうがよいかも

[okky (2487)]

intなのにreturn;を返しているコード

呼び出し側を調べたほうがよいかと。

呼び出している側で、戻り値を使っていないのであれば、コメントを関数の先頭と問題の return; 文につけた上で上司に報告して放置(この場合はそもそも戻り値に意味があるのか、あるのに caller が使っていないのが悪いのか、調べる必要がでる)。

しかし、使っているのであれば危険なので直すよう主張するべき。一旦見失うと、ランダムにしか見えない戻り値がなぜ発生するのかで悩む事になりかねません。

--
fjの教祖様

Re:Callerを調べたほうがよいかも

[Anonymous Coward]

return; を自動的に return (you);に書き換えるプリプロセッサ作りませんか

Re:Callerを調べたほうがよいかも

[yuri (9976)]

コメントありがとうございます。

今回は、手で書きなおしました。
下手に自動化して中身を間違えたらマジで洒落にならない予感がするのと、そんなプリプロセッサを作るだけの器量がありません（；へ；）

#優しく教えて下さいm(__)m

Re:Callerを調べたほうがよいかも

[yuri (9976)]

コメントありがとうございます。

今回の返り値の型が違うreturn文を含むファイルをざざーっと眺めてみたところ、return(int型);という正しい書式と、return;という書式が入り交じっていたので、Aさんが最初正しい表記をしていたのに、そこに追加編集をしたBさんがコピペで型の違うreturnを貼っちゃったんじゃないかな～と思っています。

#あまりにも耐えられなかったので先輩の忠告を無視して修正しちゃいました