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

Reply via email to