You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@commons.apache.org by "Thomas Neidhart (JIRA)" <ji...@apache.org> on 2013/08/31 23:03:51 UTC

[jira] [Commented] (MATH-1001) Add Frequency#incrementValue(char|int|long) methods ?

    [ https://issues.apache.org/jira/browse/MATH-1001?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13755603#comment-13755603 ] 

Thomas Neidhart commented on MATH-1001:
---------------------------------------

Primitives should be autoboxed to their wrapper classes which implement Comparable. I do not see any compiler warnings when using primitives, which (eclipse) settings do you use?

The Frequency class should be made generic imho, see also MATH-944.
                
> Add Frequency#incrementValue(char|int|long) methods ?
> -----------------------------------------------------
>
>                 Key: MATH-1001
>                 URL: https://issues.apache.org/jira/browse/MATH-1001
>             Project: Commons Math
>          Issue Type: Improvement
>            Reporter: Sebb
>
> The Frequency class has addValue(char|int|long) methods, but the incrementValue method only supports Comparable<?>
> It would avoid casting warnings in code if primitives were supported, as well as being more consistent.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira