On 07/03/2018 01:07, KUBOTA Yuji wrote:
Nice catch! I removed this unused variable.
This looks good to me as well. I have imported your patch
and sent it to our test system and things are looking fine.
Could you prepare a final changeset with a properly formatted
comment , use jcheck  to verify that your changeset conforms
to the OpenJDK rules (no trailing whitespaces, no tabs etc...),
and upload it somewhere to http://cr.openjdk.java.net/~ykubota/8169358 ?
If there aren't any other review feedback in the mean time, I'll
be happy to sponsor your changes when that's ready.