Re: Reimplemented URLEncoder/Decoder

[Tom Tromey]

>>>>> "Mark" == Mark Wielaard <address@hidden> writes:

Mark>     * java/net/URLDecoder.java: Reimplement to take character encoding
Mark>     into account.
Mark>     * java/net/URLEncoder.java: Ditto

This is great.  Thanks for doing it.  I do have a couple nits to pick
though.

Mark> Even though these version do more things some of my tests were
Mark> two times faster when compiled with gcj.

Cool.

Mark>   * where XX is the hexadecimal representation of that character.  Note
Mark>   * that since unicode characters are 16 bits, and this method encodes 
only
Mark>   * 8 bits of information, the lower 8 bits of the character are used.

This part of the comment is now wrong.


Mark>       try
Mark>         {
Mark>           bytes[index] = (byte)Integer.parseInt(sub, 16);
Mark>           index++;
Mark>         }
Mark>       catch (NumberFormatException nfe)
Mark>         {
Mark>           // Ignore badly encoded char
Mark>         }

The 1.4 docs say that the implementation can either leave illegal
characters alone or it can throw IllegalArgumentException, but this
piece of code seems to do neither.  Maybe I'm misunderstanding the
docs?  Or maybe we should throw IllegalArgumentException here?

Tom