リファクタリング中に見つけた駄目コード

また人様のソースを引き継いでしまいました。これを書いた人は while(1) が好きみたいで、あちこちに埋め込んであるのです。どれぐらい好きかというと、こんなコードを見つけちゃったぐらい。

while(1)
{
    if( hoge == 1 )
    {
        break ;
    }
}

意味は分かるけど、正直どうかと思うよね…

それに加えて while(1) はコンパイラさんが「定数値使うな」とインフォメーションを出してくるのであまり好きじゃない。そんなわけでSubversionで分岐したあと、せっせとwhile文を書き直して一人ソースチェックまでやるのですが、本線に戻す勇気が無くて放置してあったり。ユニットテスト出来れば安心だけど、俺様ソースチェックだけじゃイマイチ自身がないのよね。