#44834: Lua: allow create_unit_full() to place a unit in a transport it can't freely load
Open Date: 2022-06-14 23:08
Last Update: 2022-06-18 14:04
URL for this Ticket:
https://osdn.net//projects/freeciv/ticket/44834
RSS feed for this Ticket:
https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=44834
---------------------------------------------------------------------
Last Changes/Comment on this Ticket:
2022-06-18 14:04 Updated by: cazfi
Comment:
Reply To ihnatus
Reply To cazfi
As 'placed' is only used within assert, I think it will cause "set, but unused variable" -warning on FREECIV_NDEBUG build.
What you think should I do, use #ifdef FREECIV_NDEBUG preprocessor branching? I likely have used place_unit() that way in some previous patch.
That's what we use in other places. A lot of those went in recently in #43868.
---------------------------------------------------------------------
Ticket Status:
Reporter: ihnatus
Owner: (None)
Type: Feature Requests
Status: Open
Priority: 5 - Medium
MileStone: S3_1 d3f
Component: Scripting API
Severity: 5 - Medium
Resolution: None
---------------------------------------------------------------------
Ticket details:
There is no problem if a unit can't load its ttttransport in the place it is. I would mark it a bug and target to 3.0 but without #44738 it's too difficult to do.
--
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/44834
RSS feed for this Ticket:
https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=44834