convert: Drop broken code to charge security unit mobility
authorMarkus Armbruster <armbru@pond.sub.org>
Sun, 1 Feb 2015 16:48:58 +0000 (17:48 +0100)
committerMarkus Armbruster <armbru@pond.sub.org>
Mon, 2 Mar 2015 07:20:51 +0000 (08:20 +0100)
commit39884aff595c44a03df2e1d3c7df8cd8ec731f80
tree854d1845d530ae714a0eab08b4c9ef7bef32f525
parent2575ea89402c8a32f1f30e85d2b6b2db21fe9970
convert: Drop broken code to charge security unit mobility

Conversion is easier when land units with capability security are
present.  Each such land unit is charged 10 mobility.  The mobility
charge is undocumented.

Land unit mobility is charged even when conversion turns out to be
impossible, say because the sector has no mobility.  I call this a
bug.  Has been that way since security land units were added in
Chainsaw 3.

Except the mobility charge doesn't actually work anymore: the changed
land unit is never written back.  Broken in commit 82c9166, v4.3.16.
Fix this bug would be trivial, but would bring back the bug described
above, and fixing that one is harder, and doesn't feel worthwhile.

Remove the broken charging of land unit mobility instead.

Signed-off-by: Markus Armbruster <armbru@pond.sub.org>
src/lib/commands/conv.c