[maven] 24/30: Add a boolean to the setCanResume method, which denotes whether a build can be resumed or not.

[rf...@apache.org]

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

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

commit 17eba3e1da48b768222bcf72b54e08589d7a87d2
Author: Martin Kanters <Ma...@infosupport.com>
AuthorDate: Fri Jun 5 21:29:17 2020 +0200

    Add a boolean to the setCanResume method, which denotes whether a build can be resumed or not.
---
 maven-core/src/main/java/org/apache/maven/DefaultMaven.java         | 6 ++----
 .../org/apache/maven/execution/DefaultMavenExecutionResult.java     | 4 ++--
 .../main/java/org/apache/maven/execution/MavenExecutionResult.java  | 5 +++--
 3 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/maven-core/src/main/java/org/apache/maven/DefaultMaven.java b/maven-core/src/main/java/org/apache/maven/DefaultMaven.java
index 1543591..5c7c9ea 100644
--- a/maven-core/src/main/java/org/apache/maven/DefaultMaven.java
+++ b/maven-core/src/main/java/org/apache/maven/DefaultMaven.java
@@ -378,10 +378,8 @@ public class DefaultMaven
                     {
                         try
                         {
-                            if ( buildResumer.persistResumptionData( result, rootProject ) )
-                            {
-                                result.setCanResume();
-                            }
+                            boolean persistenceResult = buildResumer.persistResumptionData( result, rootProject );
+                            result.setCanResume( persistenceResult );
                         }
                         catch ( BuildResumptionPersistenceException e )
                         {
diff --git a/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionResult.java b/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionResult.java
index ae87d5c..ecddd66 100644
--- a/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionResult.java
+++ b/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionResult.java
@@ -118,8 +118,8 @@ public class DefaultMavenExecutionResult
     }
 
     @Override
-    public void setCanResume()
+    public void setCanResume( boolean canResume )
     {
-        this.canResume = true;
+        this.canResume = canResume;
     }
 }
diff --git a/maven-core/src/main/java/org/apache/maven/execution/MavenExecutionResult.java b/maven-core/src/main/java/org/apache/maven/execution/MavenExecutionResult.java
index 7a86afb..285aab0 100644
--- a/maven-core/src/main/java/org/apache/maven/execution/MavenExecutionResult.java
+++ b/maven-core/src/main/java/org/apache/maven/execution/MavenExecutionResult.java
@@ -76,9 +76,10 @@ public interface MavenExecutionResult
     boolean canResume();
 
     /**
-     * Indicate that the build could be resumed by a second invocation of Maven.
+     * Indicate that the build can or cannot be resumed by a second invocation of Maven.
+     * @param canResume <code>true</code> when it is possible to resume the build, <code>false</code> otherwise.
      * @see org.apache.maven.execution.BuildResumer
      * @see #canResume()
      */
-    void setCanResume();
+    void setCanResume( boolean canResume );
 }