I eyeballed quickly and it looks ok.

NonSerializableTest
--
  59         return new String[][][] {
  60             // Write NonSerial1, Read NonSerial1
  61             new String[][] {new String[] {"NonSerialA_1", "-cp", ".", 
"TestEntry", "-s", "A"}},
  62             new String[][] {new String[] {"NonSerialA_1", "-cp", ".", 
"TestEntry", "-d"}},

You don’t need declare the allocations within array initializer blocks.

Paul.

> On 4 Jun 2017, at 18:37, Hamlin Li <[email protected]> wrote:
> 
> Ping.
> 
> Thank you
> 
> -Hamlin
> 
> 
> On 2017/6/1 16:22, Hamlin Li wrote:
>> Would you please review the below patch?
>> 
>> bug: https://bugs.openjdk.java.net/browse/JDK-8180927
>> 
>> webrev: http://cr.openjdk.java.net/~mli/8180927/webrev.00/
>> 
>> Thank you
>> 
>> -Hamlin
>> 
> 

Reply via email to