is_action_possible(): Redundant checks for CLEAN_*

• Informador: cazfi
• Propietario: cazfi
• Tipo: Bugs
• Estado: Cerrado
• Componente: General
• Hito: 3.0.6 (cerrado)
• Prioridad: 5 - Medium
• Gravedad: 5 - Medium
• Resolución: Fixed
• Fichero: 2

Details

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.

Ticket History (3/5 Histories)

2022-11-27 03:18 Updated by: cazfi

• New Ticket "is_action_possible(): Redundant checks for CLEAN_*" created

2022-12-17 22:40 Updated by: cazfi

• File 0031-Remove-redundant-checks-for-cleaning-actions.patch (File ID: 11142) is attached

2022-12-17 22:41 Updated by: cazfi

• File 0016-Remove-redundant-checks-for-cleaning-actions.patch (File ID: 11143) is attached

2022-12-17 22:42 Updated by: cazfi

• Propietario Update from (Ninguno) to cazfi
• Resolución Update from Ninguno to Accepted
• Hito Update from (Ninguno) to 3.0.6 (cerrado)

Comentario

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 Updated by: cazfi

• Estado Update from Open to Cerrado
• Resolución Update from Accepted to Fixed