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

svn commit: r809130 - /maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/DefaultMavenReportExecutor.java

Author: olamy
Date: Sat Aug 29 13:15:30 2009
New Revision: 809130

URL: http://svn.apache.org/viewvc?rev=809130&view=rev
Log:
code cleanup

Modified:
    maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/DefaultMavenReportExecutor.java

Modified: maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/DefaultMavenReportExecutor.java
URL: http://svn.apache.org/viewvc/maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/DefaultMavenReportExecutor.java?rev=809130&r1=809129&r2=809130&view=diff
==============================================================================
--- maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/DefaultMavenReportExecutor.java (original)
+++ maven/plugins/branches/maven-site-plugin-3.x/src/main/java/org/apache/maven/plugins/site/DefaultMavenReportExecutor.java Sat Aug 29 13:15:30 2009
@@ -70,20 +70,12 @@
         List<String> imports = new ArrayList<String>();
 
         imports.add( "org.apache.maven.reporting.MavenReport" );
-        // imports.add( "org.apache.maven.reporting.AbstractMavenReport" );
         imports.add( "org.apache.maven.doxia.siterenderer.Renderer" );
         imports.add( "org.apache.maven.doxia.sink.SinkFactory" );
         imports.add( "org.codehaus.doxia.sink.Sink" );
         imports.add( "org.apache.maven.doxia.sink.Sink" );
         imports.add( "org.apache.maven.doxia.sink.SinkEventAttributes" );
 
-        /*
-         * imports.add( "org.codehaus.plexus.util.xml.Xpp3Dom" ); imports.add(
-         * "org.codehaus.plexus.util.xml.pull.XmlPullParser" ); imports.add(
-         * "org.codehaus.plexus.util.xml.pull.XmlPullParserException" ); imports.add(
-         * "org.codehaus.plexus.util.xml.pull.XmlSerializer" );
-         */
-
         RepositoryRequest repositoryRequest = new DefaultRepositoryRequest();
         repositoryRequest.setLocalRepository( mavenReportExecutorRequest.getLocalRepository() );
         repositoryRequest.setRemoteRepositories( mavenReportExecutorRequest.getMavenSession().getRequest().getRemoteRepositories() );
@@ -182,11 +174,6 @@
             getLog().warn( "skip ClassCastException " + e.getMessage() );
             return null;
         }
-        catch ( Throwable e )
-        {
-            getLog().error( "error configuring mojo " + mojoExecution.toString() + " : " + e.getMessage(), e );
-            throw e;
-        }
     }
 
     private boolean isMavenReport( MojoExecution mojoExecution, PluginDescriptor pluginDescriptor )