On Tue, 16 Dec 2025 16:31:18 GMT, Alexey Semenyuk <[email protected]> wrote:

>> - Add exception type as an additional generic parameter to all `Throwing...` 
>> interfaces
>> - Add unit tests for "jdk.jpackage.internal.util.function" package
>> - Add `ExceptionBox.toUnchecked()`. It is equivalent to 
>> `ExceptionBox.rethrowUnchecked()`, but doesn't throw
>> - Replace `ExceptionBox.rethrowUnchecked(...)` with 
>> `ExceptionBox.toUnchecked(...)`; this increased test coverage of 
>> "jdk.jpackage.internal.util.function" package from 86% to 97%
>> - Change exception handling in `ExceptionBox.toUnchecked()` (former 
>> `ExceptionBox.rethrowUnchecked()`):
>>   -  `InterruptedException`: instead of rethrowing an exception of this 
>> type, it calls `Thread.currentThread().interrupt()` and then rethrows it
>>  - Add `ExceptionBox.unbox()` (former `TKit.unbox()`). It is complementary 
>> to `ExceptionBox.toUnchecked()`
>>  - Catch `Exception` instead of `Throwable` where appropriate to avoid 
>> catching `Error`-s. The idea is not to handle fatal errors
>>  - Replace vague `throws Throwable` exception specifications with more 
>> specific ones where appropriate
>>  - Add `ExceptionBox.reachedUnreachable()` for use in locations that control 
>> flow is not expected to reach
>> 
>> Supplementary changes:
>>  - Take advantage of the updated `Throwing...` interfaces in the `Result` 
>> class.
>>  - Add unit tests for the `Result` class and fix uncovered minor issues.
>
> Alexey Semenyuk has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Fix whitespaces

Marked as reviewed by almatvee (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/28731#pullrequestreview-3584975155

Reply via email to