Adjust city name in network protocol

• Relator: cazfi
• Dono: (Nenhum)
• Tipo: Patches
• Estado: Aberto
• Componente: General
• Marcos: S3_3 npf
• Prioridade: 5 - Medium
• Gravidade: 5 - Medium
• Resolução: Nenhum
• Arquivo: Nenhum

Details

Likely the network protocol could be made better concerning city names (in city_info etc), esp. after #45786 and #45844

Ticket History (3/4 Histories)

2022-10-13 02:50 Updated by: cazfi

• New Ticket "Adjust city name in network protocol" created

2022-10-27 01:54 Updated by: cazfi

• Marco Update from S3_1 npf (fechado) to S3_2 npf

Comentário

Network protocol is already smart enough to send city name as NULL-terminated string, and not a fixed size array. So nothing to do for that.

Second thing considered was that city name could be moved to a packet of its own to avoid too big PACKET_CITY_INFO. For that part, there's a couple of much worse offenders -> will open a new tickets about those.
City name as part of PACKET_CITY_INFO isn't problem in S3_1. Let's recheck the situation before 3.2, especially if MAX_LEN_CITYNAME gets increased -> retargeting to S3_2-npf

2022-11-21 10:50 Updated by: cazfi

Comentário

Reply To cazfi

Second thing considered was that city name could be moved to a packet of its own to avoid too big PACKET_CITY_INFO. For that part, there's a couple of much worse offenders -> will open a new tickets about those.

-> #46079, #46080

(Edited, 2022-11-21 11:08 Updated by: cazfi)

2024-10-19 00:13 Updated by: cazfi

• Marco Update from S3_2 npf to S3_3 npf