kwin commented on code in PR #872:
URL: https://github.com/apache/maven/pull/872#discussion_r1023128378
##########
maven-core/src/main/java/org/apache/maven/internal/impl/PropertiesAsMap.java:
##########
@@ -71,6 +71,8 @@ public Entry<String, String> next()
{
throw new NoSuchElementException();
}
+ Entry<Object, Object> next = this.next;
+ this.next = null;
Review Comment:
This should only be null once the final property key has been reached, there
is no obligation to call `hasNext()` in between, e.g. something like two
subsequent calls to `next()` must succeed in case the underlying property
contains more than one key.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
