is_action_possible(): Redundant checks for CLEAN_*

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

詳細

When is_action_possible() determines target extra for CLEAN_POLLUTION or CLEAN_FALLOUT by prev_extra_in_tile() some of the later checks are redundant (should be done only for caller-provided extras); we already know that the extra is on tile, and that it's removed by the action.

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

2022-11-27 03:18 更新者: cazfi

• 新しいチケット "is_action_possible(): Redundant checks for CLEAN_*" が作成されました

2022-12-17 22:40 更新者: cazfi

• 添付ファイル 0031-Remove-redundant-checks-for-cleaning-actions.patch (File ID: 11142) が付加されました

2022-12-17 22:41 更新者: cazfi

• 添付ファイル 0016-Remove-redundant-checks-for-cleaning-actions.patch (File ID: 11143) が付加されました

2022-12-17 22:42 更新者: cazfi

• 担当者 が (未割り当て) から cazfi に更新されました
• 解決法 が なし から 受領 に更新されました
• マイルストーン が (未割り当て) から 3.0.6 (完了済み) に更新されました

コメント

Same logic (existing code & how to fix it) applies to S3_0, though there it's not about action, but activity.

2022-12-23 22:07 更新者: cazfi

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