On Mon, 20 Nov 2023 12:44:12 GMT, Daniel Fuchs <[email protected]> wrote:
> That's a good point. I see that there's another `decode(String, String)` > method above in this file that has the same old `@implNote` but not > `@throws`. Maybe the implNote should be removed there too and the `@throws` > added. Not sure it's worth touching the first `@Deprecated decode(String)` > method though. Opinions? Since we are editing this method descriptions then it's probably best to add the throws IAE to the other 2-arg decode method. I suppose the 1-arg/deprecated decode method should document the exception too, doesn't need to be done in this PR of course. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16709#discussion_r1399179940
