Author: sveinung Date: Thu Oct 6 14:05:31 2016 New Revision: 34019 URL: http://svn.gna.org/viewcvs/freeciv?rev=34019&view=rev Log: savegame3: add all actions to order validation.
The unit action order target validation code didn't handle all actions. Have it handle all currently existing actions. Make it object when a new action is added but not handled. See bug #25152 Modified: trunk/server/savegame3.c Modified: trunk/server/savegame3.c URL: http://svn.gna.org/viewcvs/freeciv/trunk/server/savegame3.c?rev=34019&r1=34018&r2=34019&view=diff ============================================================================== --- trunk/server/savegame3.c (original) +++ trunk/server/savegame3.c Thu Oct 6 14:05:31 2016 @@ -5460,7 +5460,7 @@ unitstr, j); if (order->order == ORDER_PERFORM_ACTION) { - switch (order->action) { + switch ((enum gen_action)order->action) { case ACTION_SPY_TARGETED_SABOTAGE_CITY: /* Sabotage target is production (-1) or a building. */ if (!(order_tgt - 1 == -1 @@ -5506,6 +5506,15 @@ case ACTION_NUKE: case ACTION_DESTROY_CITY: case ACTION_EXPEL_UNIT: + case ACTION_RECYCLE_UNIT: + case ACTION_DISBAND_UNIT: + case ACTION_HOME_CITY: + case ACTION_UPGRADE_UNIT: + case ACTION_PARADROP: + case ACTION_AIRLIFT: + case ACTION_ATTACK: + case ACTION_CONQUER_CITY: + case ACTION_HEAL_UNIT: case ACTION_COUNT: /* Target in order unsupported. */ _______________________________________________ Freeciv-commits mailing list Freeciv-commits@gna.org https://mail.gna.org/listinfo/freeciv-commits