「「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:警告がいっぱい

[Anonymous Coward]

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

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

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