[ 
https://issues.apache.org/jira/browse/LANG-495?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12683047#action_12683047
 ] 

Sebb commented on LANG-495:
---------------------------

Sorry, the annotation crept in; it's not relevant to this patch.

> Possible implementation of max/min using generics
> -------------------------------------------------
>
>                 Key: LANG-495
>                 URL: https://issues.apache.org/jira/browse/LANG-495
>             Project: Commons Lang
>          Issue Type: Improvement
>            Reporter: Sebb
>         Attachments: ObjectUtils.patch
>
>
> Attached is a patch to add generics to max and min. 
> It seems to work OK with the current tests.
> It causes a compile-time error if one tries to code e.g. 
> ObjectUtils.max(String,Integer).
> Previously that was allowed, but would cause a class cast exception at run 
> time.
> Can anyone review please?
> BTW, the tests for max/min use Comparable instead of Date - is there a good 
> reason for this? User code is likely to use the actual type in this case.
> Changing Comparable to Date fixes various raw type warnings.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to