On Mon, 28 Sep 2026 19:13:19 GMT, Andy Goryachev <[email protected]> wrote:
> Fixed to use `Objects.requireNonNull()` instead of `Objects.nonNull()`. > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). Looks good, it is indeed a trivial change so 1 reviewer should be enough. One question before I run tests and approve. modules/jfx.incubator.richtext/src/main/java/jfx/incubator/scene/control/richtext/Marker.java line 58: > 56: > 57: private Marker(TextPos pos) { > 58: Objects.nonNull(pos); Mistake while replacing, or should this check actually be removed? ------------- PR Review: https://git.openjdk.org/jfx/pull/2331#pullrequestreview-5355212966 PR Review Comment: https://git.openjdk.org/jfx/pull/2331#discussion_r4135620101
