You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by rf...@apache.org on 2020/03/30 15:38:08 UTC

[maven] 17/18: Use Stream#iterator to lazily traverse outputFiles

This is an automated email from the ASF dual-hosted git repository.

rfscholte pushed a commit to branch MNG-4660
in repository https://gitbox.apache.org/repos/asf/maven.git

commit 6023e04436b694e813d3f0beebfc53cb1346433d
Author: Maarten Mulders <ma...@infosupport.com>
AuthorDate: Sun Mar 29 21:28:46 2020 +0200

    Use Stream#iterator to lazily traverse outputFiles
---
 maven-core/src/main/java/org/apache/maven/ReactorReader.java | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/maven-core/src/main/java/org/apache/maven/ReactorReader.java b/maven-core/src/main/java/org/apache/maven/ReactorReader.java
index 13d7205..ebcf452 100644
--- a/maven-core/src/main/java/org/apache/maven/ReactorReader.java
+++ b/maven-core/src/main/java/org/apache/maven/ReactorReader.java
@@ -30,9 +30,9 @@ import java.util.Collection;
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.HashSet;
+import java.util.Iterator;
 import java.util.List;
 import java.util.Map;
-import java.util.stream.Collectors;
 import java.util.stream.Stream;
 
 import javax.inject.Inject;
@@ -250,8 +250,10 @@ class ReactorReader
             // Not using File#lastModified() to avoid a Linux JDK8 milliseconds precision bug: JDK-8177809.
             long artifactLastModified = Files.getLastModifiedTime( packagedArtifactFile.toPath() ).toMillis();
 
-            for ( Path outputFile : outputFiles.collect( Collectors.toList() ) )
+            Iterator<Path> iterator = outputFiles.iterator();
+            while ( iterator.hasNext() )
             {
+                Path outputFile = iterator.next();
                 long outputFileLastModified = Files.getLastModifiedTime( outputFile ).toMillis();
                 if ( outputFileLastModified > artifactLastModified )
                 {