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 Marked as reviewed by dfuchs (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/16709#pullrequestreview-1750640905
