Joe Schaefer wrote:
In its current form, the internal url_decode() function is a total mess.
Besides simple url-decoding, it does charset divination. It turns out to
be a lot cleaner to do the divination *after* decoding (not *during*),
so I'd like to propose a new function
apreq_charset_t apreq_divine_charset (unsigned const char *s, apr_size_t slen)
that does the same logic, but on the decoded string.
The current APIs for apreq_decode and apreq_decodev don't need to change
because of this (only one current test result needs "fixing").
I agree with the idea...
Kudos on the function name.
--
END
------------------------------------------------------------
What doesn't kill us can only make us stronger.
Nothing is impossible.
Philip M. Gollucci ([EMAIL PROTECTED]) 301.254.5198
Consultant / http://p6m7g8.net/Resume/
Senior Developer / Liquidity Services, Inc.
http://www.liquidityservicesinc.com
http://www.liquidation.com
http://www.uksurplus.com
http://www.govliquidation.com
http://www.gowholesale.com
