On 18/11/2011 18:27, Darryl Mocek wrote:
Hello. Please review this patch to fix a serialization issue with String's CASE_INSENSITIVE_ORDER. If you serialize, then deserialize the class, the equals test will fail in the comparison of what was serialized with what was deserialized. Webrev, including test, can be found here: http://cr.openjdk.java.net/~sherman/darryl/5035850/webrev

Thanks,
Darryl
This looks okay to me but I would suggest adding a comment to readResolve, maybe something like "Replaces the de-serialized object" as the causal reader may not know what this method is for.

-Alan.

Reply via email to