Hi Yuji,
On 07/03/2018 01:07, KUBOTA Yuji wrote:
Nice catch! I removed this unused variable.
http://cr.openjdk.java.net/~ykubota/8169358/webrev.04/
Best regards
Yuji
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 [1], use jcheck [2] 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.
best regards,
-- daniel
[1] http://openjdk.java.net/guide/producingChangeset.html
[2] http://openjdk.java.net/projects/code-tools/jcheck/
