> 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/5af3006b..c09d4011

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

  Stats: 27 lines in 1 file changed: 2 ins; 12 del; 13 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

Reply via email to