読んでいて、微妙にイラッとしたコード。

ret = getParam(hoge);
if (ret < 0) {
    return ERROR;
}

if (ret >= 0) {
    return OK;
}

後半は条件分岐いらねぇだろ。入れるとしてもelseだろ。

どうでも良いだけに余計にイラッとする感じです。