On Tue, 29 Sep 2026 16:03:01 GMT, Lukasz Kostyra <[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). > > 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? not needed - the null check happens in the caller (I should have left a comment in the PR) ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/2331#discussion_r4135693742
