Don't let designate check total cost before doing anything
authorMarkus Armbruster <armbru@pond.sub.org>
Sun, 17 Feb 2008 07:47:23 +0000 (08:47 +0100)
committerMarkus Armbruster <armbru@pond.sub.org>
Sun, 17 Feb 2008 07:47:23 +0000 (08:47 +0100)
commit8227d8c8efe6e90c1a8afc4cc3c1c3c5d95becd2
tree7e15ffdc5c1b8a3707cbe46be591fd23403a3c9a
parent6110da1ef45575c8fe27ac51e7f436fdb2ab4c00
Don't let designate check total cost before doing anything

desi() ran the designate code twice, first for adding up the cost,
then for changing designation.  However, the checking pass already
changed the sector when that cost nothing.  The checking pass also
suppressed messages.  There was at least one message that never got
printed because it was suppressed in the checking pass, and the
condition for it was no longer true in the changing pass, due to the
premature sector change: when a deity changed a non-coastal sector to
harbor or bridge head.

The total cost check is of limited value: designate costing money is a
bad idea, and the stock game has no such sectors.  Not enough value to
justify keeping and fixing this disgusting mess.  Remove it instead.
src/lib/commands/desi.c