There's also inconsistent use when returning the channel name to
the user.  The old way uses 'chptr->chname' and the new way uses
'name'.  Just something else to nitpick.

- Chris


On Tue, Jan 23, 2007 at 02:18:02AM -0500, Kevin L. Mitchell wrote:
> On Mon, 2007-01-22 at 22:15 -0500, Jeannot Langlois wrote:
> > Agreed - It's always better to avoid hardcoding.
> 
> Sorry to nitpick, but there's a couple of important things left out of
> your patch--all features need to be documented in doc/readme.features,
> and you should also list the feature in the appropriate place in
> doc/example.conf.  Your patch should also include a ChangeLog hunk...
> -- 
> Kevin L. Mitchell <[EMAIL PROTECTED]>
> Undernet Coder Committee, co-maintainer



> _______________________________________________
> Coder-com mailing list
> Coder-com@undernet.org
> http://undernet.sbg.org/mailman/listinfo/coder-com


-- 
Chris Behrens
Senior Software Architect
XO Communications
_______________________________________________
Coder-com mailing list
Coder-com@undernet.org
http://undernet.sbg.org/mailman/listinfo/coder-com

Reply via email to