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

Stefan Karlsson stefank at openjdk.org
Fri Jan 5 09:10:36 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.

This pull request has now been integrated.

Changeset: 868f8745
Author:    Stefan Karlsson <stefank at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/868f8745faf70c915d8294ae8f85b2d6aa096900
Stats:     6 lines in 1 file changed: 0 ins; 2 del; 4 mod

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

Reviewed-by: dholmes, lmesnik

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

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


More information about the core-libs-dev mailing list