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

Make prewrite callbacks more robust

Browse source 
Treat zero owner just like efficiency below minimum.  Before, cargo
was taken off carriers only when efficiency fell below minimum, not
when owner changed to zero.

Run item_prewrite() unconditionally, for simplicity.
This commit is contained in:
Markus Armbruster 2008-09-13 13:21:12 -04:00
parent fc42f66b4a
commit 861a66625c
4 changed files with 18 additions and 20 deletions
Download patch file Download diff file Expand all files Collapse all files

11
src/lib/subs/land.c
View file

@ -63,14 +63,13 @@ lnd_prewrite(int n, void *old, void *new)
{
struct lndstr *oldlp = old;
struct lndstr *lp = new;
natid own = lp->lnd_own;
natid own = lp->lnd_effic < LAND_MINEFF ? 0 : lp->lnd_own;
if (lp->lnd_own && lp->lnd_effic < LAND_MINEFF) {
own = 0;
if (!own) {
lp->lnd_effic = 0;
lp->lnd_ship = lp->lnd_land = -1;
} else {
item_prewrite(lp->lnd_item);
}
item_prewrite(lp->lnd_item);
if (CANT_HAPPEN(lp->lnd_ship >= 0 && lp->lnd_land >= 0))
lp->lnd_land = -1;
@ -79,7 +78,7 @@ lnd_prewrite(int n, void *old, void *new)
if (oldlp->lnd_land != lp->lnd_land)
lnd_carrier_change(lp, EF_LAND, oldlp->lnd_land, lp->lnd_land);
/* We've avoided assigning to lp->lnd_own, in case oldsp == sp */
/* We've avoided assigning to lp->lnd_own, in case oldlp == lp */
if (oldlp->lnd_own != own)
lost_and_found(EF_LAND, oldlp->lnd_own, own,
lp->lnd_uid, lp->lnd_x, lp->lnd_y);