On Tue, 26 Jul 2022 21:14:58 GMT, Andy Goryachev <[email protected]> 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