#45567: generate_packets.py: Support for opposite of handle-via-packet
Open Date: 2022-09-04 06:25
Last Update: 2022-09-06 01:24
URL for this Ticket:
https://osdn.net//projects/freeciv/ticket/45567
RSS feed for this Ticket:
https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=45567
---------------------------------------------------------------------
Last Changes/Comment on this Ticket:
2022-09-06 01:24 Updated by: alienvalkyrie
* Resolution Update from Accepted to None
Comment:
I think completely phasing out handle-via-fields (apart from the empty packet special case, if you wanna call it that) is actually the way to go. Reason being; if it already was that way, I doubt anyone would seriously consider it worth adding. So unless someone considers handle-via-fields an important feature, I'm going to be doing that.
---------------------------------------------------------------------
Ticket Status:
Reporter: cazfi
Owner: alienvalkyrie
Type: Patches
Status: Open [Owner assigned]
Priority: 5 - Medium
MileStone: 3.2.0
Component: Bootstrap
Severity: 5 - Medium
Resolution: None
---------------------------------------------------------------------
Ticket details:
If this feature already exist, it's not obvious from documentation in packets.def.
Sometimes it's annoying in development when you add a parameter to a packet, temporarily, for debugging purposes, and suddenly the handle function prototype changes (as parameter count goes over 5). Then you have to adjust all the code for that, and your two-line debug test comes much bigger diff to handle. For these cases it would be handy if one could tell generate_packets.py to just add new parameter to the handle -function despite their count going over 5.
--
Ticket information of Freeciv project
Freeciv Project is hosted on OSDN
Project URL: https://osdn.net/projects/freeciv/
OSDN: https://osdn.net
URL for this Ticket:
https://osdn.net/projects/freeciv/ticket/45567
RSS feed for this Ticket:
https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=45567