Looks fine.
-Sherman
On 10/01/2013 11:53 AM, roger riggs wrote:
Ping, needs a Reviewer.
http://cr.openjdk.java.net/~rriggs/webrev-serial-refactor-8024896/
Updated the webrev with the renamed test classes to distinguish them from the
non-serialization tests in a different package.
Thanks,
Ping, needs a Reviewer.
http://cr.openjdk.java.net/~rriggs/webrev-serial-refactor-8024896/
Updated the webrev with the renamed test classes to distinguish them
from the
non-serialization tests in a different package.
Thanks, Roger
On 9/23/2013 11:04 AM, Stephen Colebourne wrote:
The
The change looks fine AFAICT. I would prefer to see the new serial
package test classes have names that end in Serialization, eg
TCKLocalDateSerialization, but if that does not happen it is not the
end of the world.
Stephen
On 18 September 2013 22:31, roger riggs roger.ri...@oracle.com wrote:
A
A review of the TCK tests for java.time resulted in a number of changes
including refactoring the serialization tests into subpackages to follow
JCK conventions,
discovery that a number of tests were missing for serialization, and cleanup
of the test code to utilize common test functions for