[Review request] 8088853: Applying dashed border to separator makes whole application hang

Jim Graham james.graham at oracle.com
Tue Feb 16 22:22:49 UTC 2016


[I believe you sent this to the wrong mailing list - the "-request" list 
is for changing your membership to the mailing list, not for sending 
mail to the list itself.  I fixed it here in my reply...]

That looks fine.  I'll note that the "dashCumulative" variable does not 
need to be computed if you don't recompute the gapLength at line 1580, 
so that line (#1578) can be moved inside the if statement.

Other than that, it looks good...

			...jim

On 2/15/2016 2:50 AM, Andrey Rusakov wrote:
> Hello, everyone! Could you look at my fix for JDK-8147944?
> http://cr.openjdk.java.net/~arusakov/8088853/webrev.00


More information about the openjfx-dev mailing list