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

Paolo Castagna commented on JENA-233:
-------------------------------------

Ack. 

However, giving people the opportunity to experience the process of checkout 
sources, generating a patch, run the tests and submitting the patch (even if 
trivial) is IMHO a good thing to do. I am happy to pay the overhead of the 
patch submission, if that is helping other developers to come closer to the 
project and, why not, in future become more active and perhaps new committers.

But, I agree with you, this was trivial.


                
> Wrong returned value for INF and -INF floats
> --------------------------------------------
>
>                 Key: JENA-233
>                 URL: https://issues.apache.org/jira/browse/JENA-233
>             Project: Apache Jena
>          Issue Type: Bug
>          Components: Jena
>    Affects Versions: Jena 2.7.0
>            Reporter: Valentin Grouès
>            Assignee: Andy Seaborne
>            Priority: Minor
>             Fix For: Jena 2.7.1
>
>   Original Estimate: 10m
>  Remaining Estimate: 10m
>
> File: com/hp/hpl/jena/datatypes/xsd/impl/XSDFloat.java
> Problem in the parsing of INF and -INF float value.
> INF is transformed to NEGATIVE_INFINITY while -INF is transformed to 
> POSITIVE_INFINITY. It should be the other way around.
>  
> current version:
>  public Object parseValidated(String lex) {
>         if (lex.equals("INF")) {
>             return new Float(Float.NEGATIVE_INFINITY);
>         } else if (lex.equals("-INF")) {
>             return new Float(Float.POSITIVE_INFINITY);
>         } else if (lex.equals("NaN")) {
>             return new Float(Float.NaN);
>         } else {
>             return Float.valueOf(lex);
>         }
>     }
>      

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira


Reply via email to