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"). -- Joe Schaefer
