On Thu, 1 Aug 2024 22:45:43 GMT, Andy Goryachev <ango...@openjdk.org> wrote:
>> Michael Strauß has updated the pull request incrementally with one >> additional commit since the last revision: >> >> fixed a bug > > modules/javafx.graphics/src/main/java/javafx/scene/Node.java line 9184: > >> 9182: * >> 9183: * @param metadata the CSS metadata of the property >> 9184: * @param result the map into which results are stored, should >> be {@code null} > > same comment, `could be null`? or perhaps remove it by adding a private method with the second argument? what do you think? ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1522#discussion_r1701996909