Re: Autoloot code:

From: Ryan (ed209@dchan.tor.hookup.net)
Date: 07/10/96


On Tue, 9 Jul 1996, ed wrotniewski wrote:

> > 
> > could anybody please tell me what seems to be wrong with these lines ?
> > 
> > if (IS_NPC(victim) && !IS_NPC(ch) && PRF_FLAGGED(ch, PRF_AUTOLOOT)
> >     command_interpreter(ch, "get all corpse");
> > 
> 
> and BTW... pulling command interpreter is a crappy way to go about that.

First of all, what do you mean by 'pulling'?  Do you perhaps mean calling 
a function?  If this is the case, then you may be in your wrong.  
command_interpreter is quite a high level function, yes, however it also 
checks positions and other such statements that help sanity within the 
code and how it performs.  I suggest rewriting the whole command 
interpreter if you already haven't.  The way it has been defaultly setup 
in circlemud code is very un-workable.

> do_get(ch, " all corpse", 0, 0); (I think that's the correct format)
> is much more cpu efficient.

If you wanted to be 'more cpu efficient', you'd probably want to get rid 
of that space in front of 'all corpse' in your argument field.


				-- Ryan



This archive was generated by hypermail 2b30 : 12/07/00 PST