is_action_possible(): Redundant checks for CLEAN_*

• Rapporteur: cazfi
• Propriétaire: cazfi
• Type: Bogues
• État: Atteints
• Composant: General
• Jalon: 3.0.6 (atteints)
• Priorité: 5 - moyen
• Sévérité: 5 - moyen
• Résolution: Fixed
• Fichier: 2

Détails

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

• Propriétaire Update from (Aucun) to cazfi
• Résolution Update from Aucun to Accepted
• Jalon Update from (Aucun) to 3.0.6 (atteints)

Commentaire

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

• État Update from Ouvert to Atteints
• Résolution Update from Accepted to Fixed