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

Replace trade_check_item_ok() by check_obj_ok()

Browse source 
Relaxes the sanity check of the argument's ef_type.  Could be avoided,
but not worth the bother.
This commit is contained in:
Markus Armbruster 2012-07-01 12:13:36 +02:00
parent 29f6d10a34
commit f75544ee70
3 changed files with 2 additions and 19 deletions
Download patch file Download diff file Expand all files Collapse all files

1
include/trade.h
View file

@ -58,7 +58,6 @@ struct trdstr {
};
extern int trade_check_ok(struct trdstr *, union empobj_storage *);
extern int trade_check_item_ok(union empobj_storage *);
extern char *trade_nameof(struct trdstr *, union empobj_storage *);
extern int trade_desc(struct trdstr *, union empobj_storage *);
extern int trade_getitem(struct trdstr *, union empobj_storage *);

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

@ -93,7 +93,7 @@ set(void)
trade_nameof(&trade, &item), ni.cur);
if (!(p = getstarg(player->argp[3], prompt, buf)))
return RET_FAIL;
if (!trade_check_item_ok(&item))
if (!check_obj_ok(&item.gen))
return RET_FAIL;
if ((price = atoi(p)) < 0)
continue;

18
src/lib/subs/trdsub.c
View file

@ -53,23 +53,7 @@
int
trade_check_ok(struct trdstr *tp, union empobj_storage *tgp)
{
return check_trade_ok(tp) && trade_check_item_ok(tgp);
}
int
trade_check_item_ok(union empobj_storage *tgp)
{
if (tgp->gen.ef_type == EF_LAND)
return check_land_ok(&tgp->land);
if (tgp->gen.ef_type == EF_PLANE)
return check_plane_ok(&tgp->plane);
if (tgp->gen.ef_type == EF_SHIP)
return check_ship_ok(&tgp->ship);
if (tgp->gen.ef_type == EF_NUKE)
return check_nuke_ok(&tgp->nuke);
CANT_REACH();
pr("Trade lot went bad!\n");
return 0;
return check_trade_ok(tp) && check_obj_ok(&tgp->gen);
}
char *