On Tue, 26 Jul 2022 21:14:58 GMT, Andy Goryachev <ango...@openjdk.org> wrote:
> - replaced with exact functional equivalent (in the presence of exceptions, > for example) Looks good. I left one optional comment. modules/javafx.graphics/src/main/java/com/sun/javafx/css/StyleManager.java line 1274: > 1272: > 1273: StylesheetContainer container = > platformUserAgentStylesheetContainers.get(n); > 1274: if((isSame = fname.equals(container.fname)) == > true) { Minor: since you are updating this line, maybe add a space after the `if`? (although since it is preexisting, it's ok to leave it as-is) ------------- Marked as reviewed by kcr (Lead). PR: https://git.openjdk.org/jfx/pull/851