Hi Yuji,

On 07/03/2018 01:07, KUBOTA Yuji wrote:
Nice catch! I removed this unused variable.

Best regards

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/

Reply via email to