On 12/3/17, 11:34 AM, Alan Bateman wrote:
On 01/12/2017 20:35, Joe Wang wrote:
:
I think URLDecoder.decode methods should have @throws
IllegalArgumentException. I realize it's implementation specific as
to whether IAE is thrown with bad input but given that the RI does
throw IAE then consumers of the API should be prepared for it. The
@implNote can stay and probably should be copied into the legacy
decode method too.
I added @throws IAE. On a 2nd thought, would that give no flexibility
to alternative impls as the general (class) spec had given? With this
addition, it becomes a requirement.
If the @throws description starts with "if the implementation ..."
then it should be clear that it is optional.
Thanks Alan. I made the change accordingly.
http://cr.openjdk.java.net/~joehw/jdk10/8183743/webrev/index.html
-Joe
-Alan
