RFR: 8345079: Simplify/cleanup Exception handling in RMIConnectionImpl [v2]
Kevin Walls
kevinw at openjdk.org
Fri Jan 17 19:14:13 UTC 2025
> This is a post-SecurityManager-removal cleanup, following-on from:
>
> 8344394: Remove SecurityManager and related calls from java.management.rmi
>
>
> Remove mention of "privilege" in Exception handling in RMIConnectionImpl.
> Remove wrapping of Exceptions in PrivilegedActionException.
>
> The methods that invoke an operation handle expected Exceptions, but also need to handle SecurityException and RuntimeException, to avoid their default case of wrapping an Exception in an IOException to signal a problem.
Kevin Walls has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:
- Remove unnecessary/dup catches
- Merge remote-tracking branch 'upstream/master' into 8345079_RMIConnectionImpl_excep
- unnecessary catches
- (c)
- 8345079: Simplify/cleanup Exception handling in RMIConnectionImpl
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/23072/files
- new: https://git.openjdk.org/jdk/pull/23072/files/cc4dcd0b..0f06f752
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=23072&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=23072&range=00-01
Stats: 18608 lines in 979 files changed: 9068 ins; 4285 del; 5255 mod
Patch: https://git.openjdk.org/jdk/pull/23072.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/23072/head:pull/23072
PR: https://git.openjdk.org/jdk/pull/23072
More information about the serviceability-dev
mailing list