empire/empserver
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix lmine to use correct coordinate system

Browse source 
It reported the engineer's location in the engineer's coordinate
system instead of the player's.  Fortunately, they're the same in
normal usage.  They can differ only when a deity uses a foreign
engineer.

The buggy code is also reachable from and march sub-command 'd', but
can't bite there, because even deities can't march foreign land units.
This commit is contained in:
Markus Armbruster 2010-05-16 11:07:04 +02:00
parent bedf3ae415
commit bc72ff768f
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/lib/commands/mine.c
View file

@ -152,7 +152,7 @@ landmine(void)
if (total_mines_laid == mines_wanted) { if (total_mines_laid == mines_wanted) {
pr("%s laid a total of %d mines in %s", pr("%s laid a total of %d mines in %s",
prland(&land), total_mines_laid, prland(&land), total_mines_laid,
xyas(sect.sct_x, sect.sct_y, land.lnd_own)); xyas(sect.sct_x, sect.sct_y, player->cnum));
if (!land.lnd_item[I_SHELL]) if (!land.lnd_item[I_SHELL])
pr(" but is now out of shells\n"); pr(" but is now out of shells\n");
else else
@ -163,7 +163,7 @@ landmine(void)
prland(&land), prland(&land),
land.lnd_mobil > 0 ? "shells" : "mobility", land.lnd_mobil > 0 ? "shells" : "mobility",
total_mines_laid, total_mines_laid,
xyas(sect.sct_x, sect.sct_y, land.lnd_own)); xyas(sect.sct_x, sect.sct_y, player->cnum));
} }
return RET_OK; return RET_OK;
} }