The fix looks good to me.
Thanks,
Alexandr.
On 7/11/2016 6:02 PM, Mario Torre wrote:
On Fri, Jul 8, 2016 at 10:05 AM, Alexandr Scherbatiy
<alexandr.scherba...@oracle.com> wrote:
On 7/4/2016 3:14 PM, Mario Torre wrote:
On Wed, Jun 29, 2016 at 12:40 PM, Mario Torre <neug...@redhat.com> wrote:
Ping?
Ping Ping?
Just a question. The isGtkSupported is moved to be a local variable. It
seems its value can't be changed in the 'if' condition on the line 60.
Thanks,
Alexandr.
You are perfectly right, this can't be changed.
I think I first added the block and then decided to have a local
variable, but never cleaned that bit...
Here's the (hopefully!) final webrev:
http://cr.openjdk.java.net/~neugens/8150954/webrev.07/
It includes the commit message and a jcecked export, so this is what
will hit the repos if there are no further edits!
Cheers,
Mario
