--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
> explain the  problems with escape_html before the patch went in?  thanks.

I believe the patch is useful though, in cases where the charset is not 
explicitely
specified and there's an odd character with the 8th bit set it will now do 
the
right thing. I guess a lot of US coders would fall in that situation... I 
suppose
it's faster than HTML::Entities (I haven't benchmarked it though).
So I suspect the patch will fix more situations than it breaks: if using
a single-byte non-ASCII encoding, it doesn't actually break anything, just 
adds
bloat. If using a multi-byte encoding escape_html was broken/inapplicable 
already.


--
Eric Cholet

Reply via email to