RFR: 8328953 : JEditorPane.read throws ChangedCharSetException [v4]

rjolly duke at openjdk.org
Tue Mar 26 15:03:40 UTC 2024


> ChangedCharSetException is used to amend the charset during read according to html directives. Currently it causes immediate exit of the method which in turn causes failure to load html documents with charset directives (even if the latter must not change after all). This PR restores the catch operation as it was before the use of try with resources.

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

  8328953 : JEditorPane.read throws ChangedCharSetException
  
  The fix is to add a nested `try`-block inside `try-with-resource`; all the exceptions are handled in the nested `try`; the outer `try`-with-resouces only closes the input stream.

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/17567/files
  - new: https://git.openjdk.org/jdk/pull/17567/files/f89808df..15dcec91

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

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/17567.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/17567/head:pull/17567

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


More information about the client-libs-dev mailing list