Summary: Make key_unit_diplomat_actions() dumber
Submitted by: sveinung
Submitted on: Wed 13 Aug 2014 02:53:21 PM UTC
Priority: 5 - Normal
Status: Ready For Test
Assigned to: sveinung
Discussion Lock: Any
Planned Release: 2.6.0
Before spy acting against own tile key_unit_diplomat_actions() will check that
there is a city on it. At the moment this check isn't wrong. Spy actions vs
units that aren't alone on their tile are illegal. The only other target type
is cities. It could even be said to be slightly beneficial: if a client that
would ask that a unit acts against its own tile and no city is present will
save a round trip to the server. No current client will ask.
The problem with the check is that its hard to discover. It could cause issues
if existing actions vs units are changed to no longer require the unit to
alone at its til or new actions are added. Remove it.
Date: Wed 13 Aug 2014 02:53:21 PM UTC Name:
0003-Prepare-for-ability-to-target-a-unit-that-is-t-alone.patch Size: 1kB
Reply to this item at:
Message sent via/by Gna!
Freeciv-dev mailing list