On Fri, 26 Jul 2024 13:09:19 GMT, Andrey Turbanov <aturba...@openjdk.org> wrote:
> There is implicit null check in line before. > > https://github.com/openjdk/jdk/blob/431d4f7e18369466eedd00926a5162a1461d0b25/src/java.base/share/classes/java/io/ObjectOutputStream.java#L1267-L1277 > > 'cl' can't be null after that. lgtm Hotspot would have elided this check anyway based on the previous implicit check. The review overhead is higher than the value add to the source. ------------- Marked as reviewed by rriggs (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/20351#pullrequestreview-2225694241