[ 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