upgrade_unit_order_targets(): Dead assignment of tgt_tile

• Relator: cazfi
• Dono: cazfi
• Tipo: Bugs
• Estado: Fechado
• Componente: Server
• Marcos: 3.1.0 (fechado)
• Prioridade: 5 - Medium
• Gravidade: 5 - Medium
• Resolução: Fixed
• Arquivo: 1

Details

upgrade_unit_order_targets():

/* The target is already specified in the new format. */
tgt_tile = index_to_tile(&(wld.map), order->target);
fc_assert(tgt_tile != NULL);
return;

tgt_tile is not used in NDEBUG build, and entire index_to_tile() call (with no relevant side-effect) is unnecessary in such cases.

Ticket History (3/4 Histories)

2022-09-27 12:08 Updated by: cazfi

• New Ticket "upgrade_unit_order_targets(): Dead assignment of tgt_tile" created

2022-10-01 19:17 Updated by: cazfi

• File 0038-Fix-dead-assignment-in-upgrade_unit_order_targets.patch (File ID: 10440) is attached

2022-10-01 19:17 Updated by: cazfi

• Dono Update from (Nenhum) to cazfi
• Resolução Update from Nenhum to Accepted

2022-10-08 11:06 Updated by: cazfi

• Estado Update from Aberto to Fechado
• Resolução Update from Accepted to Fixed