samabcde opened a new pull request, #300:
URL: https://github.com/apache/commons-collections/pull/300

   …y and currentValue.
   
   - The problem occur when the iterator called `hasNext` and return false, 
which set null to `currentKey`. Hence `remove` method calling 
`parent.remove(currentKey);` will not remove the current entry. 
   - Propose to fix by removing lines setting null, other than releasing the 
reference earlier, can't think of other reason to set them to null.
   - Rename variable to be more understandable.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to