commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Phil Steitz (JIRA)" <>
Subject [jira] Resolved: (MATH-258) Frequency class Javadoc is ambiguous; need more tests
Date Sun, 12 Apr 2009 19:35:14 GMT


Phil Steitz resolved MATH-258.

    Resolution: Fixed

Fixed in r764316

> Frequency class Javadoc is ambiguous; need more tests
> -----------------------------------------------------
>                 Key: MATH-258
>                 URL:
>             Project: Commons Math
>          Issue Type: Bug
>            Reporter: Sebb
>             Fix For: 2.0
> [Extracted from mailing list so it does not get lost]
> The Javadoc is ambiguous - can one add both int and char to the same instance of Frequency?
> Depending on how one reads the class Javadoc, this should be allowed,
> as one can compare "int" and "char" in the same way that one can
> compare "int" and "long" - i.e. both are "comparable". Now "int" and
> "char" are not Comparable, but then neither are "int" and "long".
> I think the Javadoc needs to make the intention clear, and then the
> tests can be enhanced to enforce the documented behaviour.
> At present, there's no check to show what happens when int and char are mixed.
> There are other tests that should be added, to show what is expected when mixing different
Comparable classes.

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

View raw message