RFR: 8318200: String::newStringNoRepl and String::getBytesNoRepl does not throw CharacterCodingException [v3]

Shaojin Wen duke at openjdk.org
Tue Oct 17 10:42:59 UTC 2023


> When calling String::newStringNoRepl and String::getBytesNoRepl, we need to use try...catch to handle CharacterCodingException and throw IllegalArgumentException instead of CharacterCodingException to make the calling code cleaner.

Shaojin Wen has updated the pull request incrementally with one additional commit since the last revision:

  Files.readString/writeString throws CCE

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/16209/files
  - new: https://git.openjdk.org/jdk/pull/16209/files/d2063b08..a93be313

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=16209&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=16209&range=01-02

  Stats: 22 lines in 3 files changed: 16 ins; 2 del; 4 mod
  Patch: https://git.openjdk.org/jdk/pull/16209.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/16209/head:pull/16209

PR: https://git.openjdk.org/jdk/pull/16209



More information about the security-dev mailing list