On Mon, 27 Nov 2023 14:24:32 GMT, Darragh Clarke <[email protected]> wrote:
>> Currently the descriptions of `URLEncoder.encode` and `URLDecoder.decode` >> don't specify their use of replacement bytes or replacement character when >> they cannot handle a character or sequence of bytes. This is longstanding >> behavior but needs to be documented. >> >> **Solution** >> - Added a new line to `URLEncoder.encode` API documentation to document that >> the charset's replacement bytes are used. >> >> - Also changed `URLDecoder.decode` API documentation to document its use of >> the charset's replacement character, also changed some wording. > > Darragh Clarke has updated the pull request incrementally with one additional > commit since the last revision: > > Implemented feedback Thanks for taking the feedback on this, I think both classes look much better now. ------------- Marked as reviewed by alanb (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/16709#pullrequestreview-1750568376
