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

Thomas Neidhart commented on MATH-627:
--------------------------------------

I failed to find any null check in the mentioned iterator, neither in the 
current trunk version nor in the version at the time the issue was created. Can 
you please specify more precisely where the problem lies?
                
> superfluously null check of SparseIterator.next()
> -------------------------------------------------
>
>                 Key: MATH-627
>                 URL: https://issues.apache.org/jira/browse/MATH-627
>             Project: Commons Math
>          Issue Type: Improvement
>            Reporter: Arne Plöse
>            Priority: Minor
>             Fix For: 3.1
>
>
> Looking at the implementation of SparseIterator in 
> OpenMapRealVector.OpenMapSparseIterator there is no chance that the entry 
> return by next() is ever null - so there is no need to chek this in nearly 
> every loop?

--
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