ソース	警告数
src/phsrot.f	1
src/qdbend.f	14
src/qdcell.f	4
src/tcsvdm.f	2
src/temit.f	3
src/tffsmatch.f	10
src/tmulta.f	4
src/tmulte.f	9
src/tmulti.f	19
src/tsolque.f	24
src/tsteee.f	2
src/tturn.f	1

現時点で既知のバグ候補

PHSROTエレメントの実装(src/phsrot.f)での配列オフセットi_p0が未定義
- i_p0はマジックナンバーであり、PHSROTの動作定義が実在しないので、原作者による修正待ち
qdcell @ src/qdcell.fの変数dcod1,dcod2,dcod3,dcod4がdetimx==0やdetimy==0の際に未初期化参照される
- 演算誤差や丸めも含めてdetimx != 0 && detimy != 0が保証されるのでなければバグである
tintraconv @ src/temit.fの変数emx1,emy1,emz1がTRPAかつINTRAな条件下でEmittance[]を呼び出すと未初期化なまま参照される。(EMITコマンドでは、一時的にTRPTフラグがクリアされているので影響しない)

あと、tturn1 @ src/tturn1.fも SOLエレメント絡みの動作が怪しそうなので、残りの調査が済んだらTrackParticles[]で実際の動作を検証してみる。

本日のツッコミ(全2件) [ツッコミを入れる]

_ 管理人さん (2008-04-15 20:40)

PHSROTのi_p0を使った箇所はbb.fのものと同じですね。i_p0=533で良いのか？

_ A. Morita (2008-04-16 16:51)

コピペか！コピペした上に、動作検証もされてないのか！ 絶望した！コピペして動作確認しないコーディング文化に絶望した！ 個人的には、動作が未定に成るコードには STOP文挿入して停止させるべきだと思う。 外部仕様書は無いので、ソースが仕様だとすれば未初期化変数を参照した動作は「未定義」なので、何が起こっても良いはず。

カテゴリー: Admin | Emacs | EPICS | Fortran | FreeBSD | GCC | hgsubversion | IPv6 | KEKB | LHC | Lisp | LLVM | MADX | Ryzen | SAD | samba | tDiary | unix | WWW | YaSAI | お仕事 | イベント | 出張 | 宴会 | 数学 | 艦これ | 買いもの | 追記 | 雑記