action_is_hostile() rework

• Informador: cazfi
• Propietario: cazfi
• Tipo: Patches
• Estado: Cerrado
• Componente: General
• Hito: 3.2.0
• Prioridad: 5 - Medium
• Gravedad: 5 - Medium
• Resolución: Fixed
• Fichero: 1

Details

action_is_hostile() has several aspects I'd like to improve.

- It lives in unittype.c, even when it's about actions
- It's another massive switch - case, based on actres. Should be turned to a quick array lookup
- Type of the parameter needs to be checked. Both actres pointer and action pointer would likely be better than current action id

Ticket History (3/5 Histories)

2023-02-20 19:51 Updated by: cazfi

• New Ticket "action_is_hostile() rework" created

2023-02-22 08:46 Updated by: cazfi

Comentario

Better to implement on top of #47464 than before it.

2023-03-02 20:29 Updated by: cazfi

• File 0028-Rename-action_is_hostile-as-actres_is_hostile.patch (File ID: 11752) is attached

2023-03-02 20:29 Updated by: cazfi

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

2023-03-04 21:29 Updated by: cazfi

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