RE: [NeoStats-Devel] [Commits] r2667 - in trunk: . modules/textserv

> Subject: [NeoStats-Devel] [Commits] r2667 - in trunk: . 
> Author: DNB
> + *    cmdparams->av[3] = optional public access on/off (default off)
> + *    cmdparams->av[4] = optional user
> + *    cmdparams->av[5] = optional host

Potential support nightmare here. Since all the above are defined as
optional, it is reasonable to expect I could skip 3 and 4 but supply 5 and
it work, however, the way the code works, in order to supply host, I must
also provide all other options. No matter how well you eventually document
the command, if you have several optional components in a command, people
will mess up and ask us before trying to work out why it doesn't work.

Since you appear to be changing TextServ to be some sort of BotServ, I
suggest you rename it accordingly and separate the bot management from the
database management as you would in a normal BotServ environment thus
providing the system you want rather than trying to fudge the TextServ
module commands to manage a BotServ system. 

Mark.