RFR: 8328953 : JEditorPane.read throws ChangedCharSetException [v3]
rjolly
duke at openjdk.org
Tue Mar 26 14:24:57 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/c67179d5..f89808df
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=17567&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=17567&range=01-02
Stats: 156 lines in 3 files changed: 78 ins; 77 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