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

Use relations_with() for getrel(NP, THEM) where NP isn't THEM

Browse source 
Replacing getrel(NP, THEM), where NP is known to be getnatp(US), by
relations_with(US, THEM) makes a difference only when US equals THEM.
Replace in places where it's obvious that they're not equal.

Adds a few calls to getnatp() hidden in relations_with().  Keeping
that optimized isn't worth it.
This commit is contained in:
Markus Armbruster 2011-02-06 10:27:45 +01:00
parent bdf63bc5fa
commit 6852ec6bc5
5 changed files with 6 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/lib/commands/cede.c
View file

@ -99,7 +99,7 @@ cede(void)
pr("You can only give to normal countries...\n");
return RET_FAIL;
}
if (getrel(natp, player->cnum) < FRIENDLY) {
if (relations_with(n, player->cnum) < FRIENDLY) {
pr("You can only cede to a country that is friendly towards you...\n");
return RET_FAIL;
}

2
src/lib/commands/decl.c
View file

@ -93,7 +93,7 @@ decl(void)
continue;
}
}
if (getrel(natp, (natid)ni.cur) == rel)
if (relations_with(who, (natid)ni.cur) == rel)
continue;
setrel(who, (natid)ni.cur, rel);
}

2
src/lib/commands/play.c
View file

@ -90,7 +90,7 @@ play_list(struct player *joe)
/* This isn't us. Can we see it? */
if (natp->nat_stat == STAT_VIS) {
/* Yes, we can see visitors are logged on */
} else if (getrel(natp, player->cnum) < ALLIED) {
} else if (relations_with(joe->cnum, player->cnum) < ALLIED) {
/* This is a non-allied country, don't show it. */
return 0;
}