Author: rfscholte
Date: Thu May  5 12:22:23 2016
New Revision: 1742421

URL: http://svn.apache.org/viewvc?rev=1742421&view=rev
Log:
rearrange methods, apply @Override

Modified:
    
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java

Modified: 
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java
URL: 
http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java?rev=1742421&r1=1742420&r2=1742421&view=diff
==============================================================================
--- 
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java
 (original)
+++ 
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java
 Thu May  5 12:22:23 2016
@@ -107,6 +107,7 @@ public abstract class AbstractRewritePom
         this.ls = ls;
     }
 
+    @Override
     public ReleaseResult execute( ReleaseDescriptor releaseDescriptor, 
ReleaseEnvironment releaseEnvironment,
                                   List<MavenProject> reactorProjects )
         throws ReleaseExecutionException, ReleaseFailureException
@@ -119,6 +120,49 @@ public abstract class AbstractRewritePom
 
         return result;
     }
+    
+    @Override
+    public ReleaseResult simulate( ReleaseDescriptor releaseDescriptor, 
ReleaseEnvironment releaseEnvironment,
+                                   List<MavenProject> reactorProjects )
+        throws ReleaseExecutionException, ReleaseFailureException
+    {
+        ReleaseResult result = new ReleaseResult();
+
+        transform( releaseDescriptor, releaseEnvironment, reactorProjects, 
true, result );
+
+        result.setResultCode( ReleaseResult.SUCCESS );
+
+        return result;
+    }
+
+    @Override
+    public ReleaseResult clean( List<MavenProject> reactorProjects )
+    {
+        ReleaseResult result = new ReleaseResult();
+
+        super.clean( reactorProjects );
+
+        if ( reactorProjects != null )
+        {
+            for ( MavenProject project : reactorProjects )
+            {
+                File pomFile = ReleaseUtil.getStandardPom( project );
+                // MRELEASE-273 : if no pom
+                if ( pomFile != null )
+                {
+                    File file = new File( pomFile.getParentFile(), 
pomFile.getName() + "." + pomSuffix );
+                    if ( file.exists() )
+                    {
+                        file.delete();
+                    }
+                }
+            }
+        }
+
+        result.setResultCode( ReleaseResult.SUCCESS );
+
+        return result;
+    }
 
     private void transform( ReleaseDescriptor releaseDescriptor, 
ReleaseEnvironment releaseEnvironment,
                             List<MavenProject> reactorProjects, boolean 
simulate, ReleaseResult result )
@@ -694,48 +738,6 @@ public abstract class AbstractRewritePom
             IOUtil.close( writer );
         }
     }
-
-    public ReleaseResult simulate( ReleaseDescriptor releaseDescriptor, 
ReleaseEnvironment releaseEnvironment,
-                                   List<MavenProject> reactorProjects )
-        throws ReleaseExecutionException, ReleaseFailureException
-    {
-        ReleaseResult result = new ReleaseResult();
-
-        transform( releaseDescriptor, releaseEnvironment, reactorProjects, 
true, result );
-
-        result.setResultCode( ReleaseResult.SUCCESS );
-
-        return result;
-    }
-
-    public ReleaseResult clean( List<MavenProject> reactorProjects )
-    {
-        ReleaseResult result = new ReleaseResult();
-
-        super.clean( reactorProjects );
-
-        if ( reactorProjects != null )
-        {
-            for ( MavenProject project : reactorProjects )
-            {
-                File pomFile = ReleaseUtil.getStandardPom( project );
-                // MRELEASE-273 : if no pom
-                if ( pomFile != null )
-                {
-                    File file = new File( pomFile.getParentFile(), 
pomFile.getName() + "." + pomSuffix );
-                    if ( file.exists() )
-                    {
-                        file.delete();
-                    }
-                }
-            }
-        }
-
-        result.setResultCode( ReleaseResult.SUCCESS );
-
-        return result;
-    }
-
     protected abstract String getResolvedSnapshotVersion( String 
artifactVersionlessKey,
                                                           Map<String, 
Map<String, String>> resolvedSnapshots );
 


Reply via email to