Re: cvs commit: modperl/t/net/perl util.pl

2002-03-28 Thread Issac Goldstand
A casual user won't understand that documentation... Hell, I'm not even sure I completely understand the implications of it and when to use/not use escape_html based on it... I think an example is called for, but not in the POD... Maybe in the Guide? Issac Eric Cholet wrote: --On

Re: cvs commit: modperl/t/net/perl util.pl

2002-03-25 Thread Eric Cholet
--On Sunday, March 24, 2002 21:57:54 + [EMAIL PROTECTED] wrote: dougm 02/03/24 13:57:53 Modified:.Changes STATUS src/modules/perl Util.xs t/net/perl util.pl Log: Submitted by: Geoff Young [EMAIL PROTECTED] Reviewed by:

Re: cvs commit: modperl/t/net/perl util.pl

2002-03-25 Thread Doug MacEachern
i had a bad feeling about this. we should not be implementing escape_html to begin with, the functionality should all be in apache. i'm going to back out the patch. anybody care to make a doc patch to explain the problems with escape_html before the patch went in? thanks.

Re: cvs commit: modperl/t/net/perl util.pl

2002-03-25 Thread Geoffrey Young
Doug MacEachern wrote: i had a bad feeling about this. we should not be implementing escape_html to begin with, the functionality should all be in apache. i'm going to back out the patch. sounds wise, especially considering people like Eric will end up with larger pages as a result,

Re: cvs commit: modperl/t/net/perl util.pl

2002-03-25 Thread Eric Cholet
--On Monday, March 25, 2002 10:29:11 -0800 Doug MacEachern [EMAIL PROTECTED] wrote: i had a bad feeling about this. we should not be implementing escape_html to begin with, the functionality should all be in apache. i'm going to back out the patch. anybody care to make a doc patch to