Patch attached.
The patch also highlights another problem, that sorting of IAtomContainer subclasses does not work with the unit test... that should be explored too...
Suggested change for the first lines of the compare method in AtomContainerComparator
public int compare(Object o1, Object o2) { // Check for nulls if (o1 == null && o2 == null) return 0; if (o1 == null) return 1; if (o2 == null) return -1;
Removed the patch which needed a small tuning, and uploaded a new version here:
https://sourceforge.net/tracker/?func=detail&aid=3413835&group_id=20024&atid=320024#
A fix based on Mark's comment below is now also attached to that patch tracker issue #3413835.
Fixed by linked patch.
Suggested change for the first lines of the compare method in AtomContainerComparator
public int compare(Object o1, Object o2) {
// Check for nulls
if (o1 == null && o2 == null)
return 0;
if (o1 == null)
return 1;
if (o2 == null)
return -1;
Removed the patch which needed a small tuning, and uploaded a new version here:
https://sourceforge.net/tracker/?func=detail&aid=3413835&group_id=20024&atid=320024#
A fix based on Mark's comment below is now also attached to that patch tracker issue #3413835.
Fixed by linked patch.