--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
