Hi, Anton.
The fix looks good overall, but a few notes:
- Can you try to change sort TimSort algorithm to mergerSort. What the order will be in this case ?(see JDK-8048887).
 - Can you run all related regression and jck tests.
Thanks.

On 18.09.2014 17:04, anton nashatyrev wrote:
Hello,
    could you please review the following fix:

fix: http://cr.openjdk.java.net/~anashaty/8058473/9/webrev.00/ <http://cr.openjdk.java.net/%7Eanashaty/8058473/9/webrev.00/>
bug: https://bugs.openjdk.java.net/browse/JDK-8058473

    Problem: DataFlavorComparator violates Comparator contract.

Fix: make the DataFlavorComparator transitive conformant (please see the bug comment for details)

Thanks!
Anton.


--
Best regards, Sergey.

Reply via email to