Changes between Initial Version and Version 1 of Ticket #145, comment 2


Ignore:
Timestamp:
Jun 7, 2011, 8:16:15 AM (13 years ago)
Author:
Peter Powers
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #145, comment 2

    initial v1  
    1616To properly compare values in AbstractParameter, the Parameter<E> Type E would have to be <E extends Comparable<E>> or better yet <E extends Comparable<? super E>> (to handle child classes). This in turn requires that all possible parameterization types implement Comparable (e.g. LocationLists, Regions etc...). It makes no sense to go to such lengths at this time.
    1717
    18  
     18Note that for Parameters compareTo is not ''consistent with equals'' as suggested in the documentation of the Comparable<E> interface. This is ok, however, Parameters should NOT be used as keys in any collections that relies on hashing or other comparisons to check or sort contents (e.g. Set, Maps etc...)