path_finding.c: Asserts comparing things to TRUE

• 報告者: cazfi
• 担当者: cazfi
• チケットの種類: バグ
• 状況: 完了
• コンポーネント: General
• マイルストーン: 3.0.2 (完了済み)
• 優先度: 5 - 中
• 重要度: 5 - 中
• 解決法: 修正済み
• ファイル: 3

詳細 返信

path_finding.c has some asserts where a value is compared to TRUE with a '=='. One should not do that as TRUE is a macro with specific value (1), and does not match other (non-zero) values though those too indicate true.

チケットの履歴 (11 件中 3 件表示)

2022-03-17 10:59 更新者: cazfi

• 新しいチケット "path_finding.c: Asserts comparing things to TRUE" が作成されました

2022-03-17 11:47 更新者: cazfi

コメント 返信

There's more of them all around the codebase when one greps for "TRUE ==", "FALSE ==", "== TRUE", "== FALSE", "TRUE !=", "FALSE !=", "!= TRUE", "!= FALSE"

Not all of them even in asserts. Limiting scope of this ticket to asserts, though.

2022-03-18 04:39 更新者: cazfi

• マイルストーン が (未割り当て) から 3.0.2 (完了済み) に更新されました

2022-04-09 04:23 更新者: cazfi

• 添付ファイル 0042-Asserts-Never-compare-boolean-value-to-exact-value-o.patch (File ID: 8955) が付加されました

2022-04-09 04:24 更新者: cazfi

• 添付ファイル 0017-Asserts-Never-compare-boolean-value-to-exact-value-o.patch (File ID: 8956) が付加されました

2022-04-09 04:24 更新者: cazfi

• 担当者 が (未割り当て) から cazfi に更新されました
• 解決法 が なし から 受領 に更新されました

2022-04-12 15:10 更新者: cazfi

• 解決法 が 受領 から なし に更新されました

コメント 返信

S3_0 version misses savegame2.c (at least) "load_success == TRUE" case.

2022-04-13 02:03 更新者: cazfi

• 添付ファイル 0009-Asserts-Never-compare-boolean-value-to-exact-value-o.patch (File ID: 8989) が付加されました

2022-04-13 02:04 更新者: cazfi

• 解決法 が なし から 受領 に更新されました

コメント 返信

Reply To cazfi

S3_0 version misses savegame2.c (at least) "load_success == TRUE" case.

- Fixed

2022-04-17 13:00 更新者: cazfi

コメント 返信

Reply To cazfi

Not all of them even in asserts. Limiting scope of this ticket to asserts, though.

Rest -> #44401

2022-04-18 12:30 更新者: cazfi

• 状況 が オープン から 完了 に更新されました
• 解決法 が 受領 から 修正済み に更新されました