Hi Paul,
Thank you for review. Modified as you suggested, it makes the test more
simple. The code was pushed.
Thank you
-Hamlin
On 2017/6/6 6:41, Paul Sandoz wrote:
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
