Philippe M. Chiasson wrote:

As usual, I usually try and complete the documentation once the code gets a go ;-)

of course :)

also continuing yesterdays story, this function doesn't know to deal with utf8 strings, right? in which case we might just as well drop it too? is there a CPAN implementation for that functionality?


I am 99.9% sure there is something on CPAN to do that

and if there is one, we should test that it does the same thing as mp1 did, and then we can consider dropping it? Geoff?


Wouldn't it be better to croak here instead?


Not really, if you try and unescape_url an undef/empty string, you get empty back.

what's empty in the Perl language? undef, ""? why "" will become undef, or the other way around?


But before you do any tweaks, let's see if we keep it at all.
I'd be more than happy to yank it out completely. Only reason I ported it forward was because it's listed in todo/release.

it's probably always a good idea to discuss first before spending time on things :)



-- __________________________________________________________________ Stas Bekman JAm_pH ------> Just Another mod_perl Hacker http://stason.org/ mod_perl Guide ---> http://perl.apache.org mailto:[EMAIL PROTECTED] http://use.perl.org http://apacheweek.com http://modperlbook.org http://apache.org http://ticketmaster.com

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to