Skip to main content

For review: finder comparator fix

2 replies [Last post]
bkurotsu
Offline
Joined: 2004-12-13
Points: 0

Simple fix to allow disabling of the sorting as the javadoc said you could. Bug id 6539180.

https://jtharness.dev.java.net/source/browse/jtharness?rev=1039&view=rev

When disabled, anything new found by the finder is simply inserted at the end of the results. Comparator is also used in TestResultTable, but that code already accepts a null value from getComparator() and just uses the default ordering in that case.

Reply viewing options

Select your preferred way to display the comments and click "Save settings" to activate your changes.
sergey_borodin
Offline
Joined: 2006-10-20
Points: 0

I'm agree with this fix.

Sergey

bkurotsu
Offline
Joined: 2004-12-13
Points: 0

Integrated into trunk and tree-int.