「「Theora1.0」 リリース」記事へのコメント

警告がいっぱい

[denchu (6847)]

試しに VisualStudio 2008 でコンパイルしてみたらものすごい数の警告が…signed と unsigned の比較や、double からの桁落ちは、まぁ、なんとかなるとして、

lib\dec\bitpack.c(36) : warning C4554: '>>' : 演算子の優先順位に問題があります。かっこを使用して優先順位を明確にしてください

さてはて、該当箇所のソースを見ると…どっちの優先順位を上げたいのでしょうね。

*_ret=(_b->ptr[0]>>7-_b->endbit)&1;

変数名からすると

*_ret=(_b->ptr[0]>>(7-_b->endbit))&1;

でしょうかね…悩ましいところです。

Re:警告がいっぱい

[taka2 (14791)]

C/C++の演算子の優先順位は、加減算の方がシフト演算子より高いですから、

*_ret=(_b->ptr[0]>>7-_b->endbit)&1;

は

*_ret=(_b->ptr[0]>>(7-_b->endbit))&1;

と解釈されます。言語仕様上は問題ない記述であり、紛らわしいからwarning を出しているにすぎません。

たとえば、最近のコンパイラは

if (fp = fopen(filename, "r")) {
    /* ファイルオープン成功時に実行 */
}

といった感じでif文の中身が代入だけだと、== の間違いじゃないかとwarningを出しますが、それと同じたぐいですね。
ある意味余計なお世話ですが、紛らわしい表現で素人がひっかかりやすいのは確かで。

Re:警告がいっぱい

[Anonymous Coward]

こういうところに3Dディスプレイなどを応用して、
「優先順位が高い順番に評価式が浮き上がって見える」というのはどうだろうか？

ほかにも深くネストした条件判断文で、「ネストすると奥に引っ込んで見える」というのだと
今どこの条件判断領域にいるのか一目でわかって良い。

コードが遠くに行き過ぎるのを見て、プログラマーにネストの深化を思いとどまらせる効果があるかもしれない。
・・・無いかもしれない。

Re:

[Anonymous Coward]

gccしか使わない人がよく書くようなコードですね。