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

Clean up useless getnatp() in conv(), nav_map(), tele()

Browse source 
Spotted by the Clang Static Analyzer.
This commit is contained in:
Markus Armbruster 2010-01-16 10:14:43 +01:00
parent 8ededa44f9
commit 8195d5fb88
3 changed files with 0 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

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

@ -52,8 +52,6 @@ conv(void)
struct lndstr land; struct lndstr land;
struct nstr_item ni; struct nstr_item ni;
natp = getnatp(player->cnum);
if (!snxtsct(&nstr, player->argp[1])) if (!snxtsct(&nstr, player->argp[1]))
return RET_SYN; return RET_SYN;
uwtoconvert = onearg(player->argp[2], "Number to convert: "); uwtoconvert = onearg(player->argp[2], "Number to convert: ");

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

@ -278,7 +278,6 @@ nav_map(int x, int y, int show_designations)
{ {
char *ptr; char *ptr;
struct nstr_sect ns; struct nstr_sect ns;
struct natstr *np;
struct sctstr sect; struct sctstr sect;
int i; int i;
/* Note this is not re-entrant anyway, so we keep the buffers /* Note this is not re-entrant anyway, so we keep the buffers
@ -305,7 +304,6 @@ nav_map(int x, int y, int show_designations)
return RET_FAIL; return RET_FAIL;
} }
snxtsct_dist(&ns, x, y, 1); snxtsct_dist(&ns, x, y, 1);
np = getnatp(player->cnum);
blankfill(wmapbuf, &ns.range, 1); blankfill(wmapbuf, &ns.range, 1);
while (nxtsct(&ns, &sect)) { while (nxtsct(&ns, &sect)) {
ptr = &wmap[ns.dy][ns.dx]; ptr = &wmap[ns.dy][ns.dx];

1
src/lib/commands/tele.c
View file

@ -46,7 +46,6 @@ tele(void)
char buf[MAXTELSIZE + 1]; /* UTF-8 */ char buf[MAXTELSIZE + 1]; /* UTF-8 */
int n; int n;
natp = getnatp(player->cnum);
if (*player->argp[0] == 'a') { if (*player->argp[0] == 'a') {
if (getele("everybody", buf) <= 0) { if (getele("everybody", buf) <= 0) {
pr("Announcement aborted\n"); pr("Announcement aborted\n");