It should not always use toString to compare. If the item is comparable, then it should use that for example. Need to think through all requirements to ensure the comparator is more fully functioned.