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

Stefan Karlsson stefank at openjdk.org
Fri Jan 5 09:10:34 UTC 2024


On Fri, 5 Jan 2024 08:22:41 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.
>
> Stefan Karlsson has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Copyright year

Thanks for the reviews. Testing with Tier1-3 passes.

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

PR Comment: https://git.openjdk.org/jdk/pull/17240#issuecomment-1878341513


More information about the hotspot-dev mailing list