On Wed, 22 Apr 2026 10:32:59 GMT, SendaoYan <[email protected]> wrote:

>> The `Assumptions.abort(...)` does indeed throw an exception, but I prefer to 
>> retain this `return` because I find it clearer when reading the code. If you 
>> have a strong preference, let me know and I can remove it.
>
> Retain the `return` is more clear when reading the code.

I suspect most IDE's will indicate that this is non-reachable

If you want to leave it, that is fine but maybe consider a comment as to why so 
a future maintainer knows why it was there

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30847#discussion_r3139228805

Reply via email to