RFR: 8322920: Some ProcessTools.execute* functions are declared to throw Throwable

Leonid Mesnik lmesnik at openjdk.org
Fri Jan 5 07:00:20 UTC 2024


On Wed, 3 Jan 2024 09:51:24 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

> Most functions in ProcessTools are declared to throw Exceptions, or one of its subclasses. However, there are a small number of functions that are unnecessarily declared to throw Throwable instead of Exception. I propose that we change them to also be declared to throw Exception.
> 
> This is a trivial patch to make it easier to refactor tests to use the updated functions.
> 
> Tested manually, but will wait for GHA to verify that the change is OK.

You need to update copyrights.

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

Marked as reviewed by lmesnik (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/17240#pullrequestreview-1805436752


More information about the core-libs-dev mailing list