Author: bentmann
Date: Sun Aug 30 18:34:46 2009
New Revision: 809399

URL: http://svn.apache.org/viewvc?rev=809399&view=rev
Log:
o Simplified code (multi-cast can be realized by a delegating listener if 
required)

Modified:
    
maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuilder.java
    
maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuilder.java
    
maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuildingRequest.java
    
maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/ModelBuildingRequest.java

Modified: 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuilder.java
URL: 
http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuilder.java?rev=809399&r1=809398&r2=809399&view=diff
==============================================================================
--- 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuilder.java
 (original)
+++ 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuilder.java
 Sun Aug 30 18:34:46 2009
@@ -18,7 +18,6 @@
 import java.io.File;
 import java.io.IOException;
 import java.util.ArrayList;
-import java.util.Arrays;
 import java.util.List;
 
 import org.apache.maven.Maven;
@@ -102,7 +101,7 @@
                 ModelBuildingRequest request = getModelBuildingRequest( 
configuration, null );
     
                 DefaultModelBuildingListener listener = new 
DefaultModelBuildingListener( projectBuildingHelper, configuration );
-                request.setModelBuildingListeners( Arrays.asList( listener ) );
+                request.setModelBuildingListener( listener );
     
                 if ( localProject )
                 {
@@ -266,7 +265,7 @@
         ModelBuildingRequest request = getModelBuildingRequest( config, null );
 
         DefaultModelBuildingListener listener = new 
DefaultModelBuildingListener( projectBuildingHelper, config );
-        request.setModelBuildingListeners( Arrays.asList( listener ) );
+        request.setModelBuildingListener( listener );
 
         request.setModelSource( new UrlModelSource( getClass().getResource( 
"standalone.xml" ) ) );
 
@@ -364,7 +363,7 @@
             request.setModelCache( modelCache );
 
             DefaultModelBuildingListener listener = new 
DefaultModelBuildingListener( projectBuildingHelper, config );
-            request.setModelBuildingListeners( Arrays.asList( listener ) );
+            request.setModelBuildingListener( listener );
 
             try
             {

Modified: 
maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuilder.java
URL: 
http://svn.apache.org/viewvc/maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuilder.java?rev=809399&r1=809398&r2=809399&view=diff
==============================================================================
--- 
maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuilder.java
 (original)
+++ 
maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuilder.java
 Sun Aug 30 18:34:46 2009
@@ -666,16 +666,13 @@
                             ModelBuildingEventCatapult catapult )
         throws ModelBuildingException
     {
-        List<ModelBuildingListener> listeners = 
request.getModelBuildingListeners();
+        ModelBuildingListener listener = request.getModelBuildingListener();
 
-        if ( !listeners.isEmpty() )
+        if ( listener != null )
         {
             ModelBuildingEvent event = new DefaultModelBuildingEvent( model, 
request, problems );
 
-            for ( ModelBuildingListener listener : listeners )
-            {
-                catapult.fire( listener, event );
-            }
+            catapult.fire( listener, event );
         }
     }
 

Modified: 
maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuildingRequest.java
URL: 
http://svn.apache.org/viewvc/maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuildingRequest.java?rev=809399&r1=809398&r2=809399&view=diff
==============================================================================
--- 
maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuildingRequest.java
 (original)
+++ 
maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuildingRequest.java
 Sun Aug 30 18:34:46 2009
@@ -61,7 +61,7 @@
 
     private ModelResolver modelResolver;
 
-    private List<ModelBuildingListener> modelBuildingListeners;
+    private ModelBuildingListener modelBuildingListener;
 
     private ModelCache modelCache;
 
@@ -271,26 +271,14 @@
         return this;
     }
 
-    public List<ModelBuildingListener> getModelBuildingListeners()
+    public ModelBuildingListener getModelBuildingListener()
     {
-        if ( modelBuildingListeners == null )
-        {
-            modelBuildingListeners = new ArrayList<ModelBuildingListener>();
-        }
-
-        return modelBuildingListeners;
+        return modelBuildingListener;
     }
 
-    public ModelBuildingRequest setModelBuildingListeners( List<? extends 
ModelBuildingListener> modelBuildingListeners )
+    public ModelBuildingRequest setModelBuildingListener( 
ModelBuildingListener modelBuildingListener )
     {
-        if ( modelBuildingListeners != null )
-        {
-            this.modelBuildingListeners = new 
ArrayList<ModelBuildingListener>( modelBuildingListeners );
-        }
-        else
-        {
-            this.modelBuildingListeners = null;
-        }
+        this.modelBuildingListener = modelBuildingListener;
 
         return this;
     }

Modified: 
maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/ModelBuildingRequest.java
URL: 
http://svn.apache.org/viewvc/maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/ModelBuildingRequest.java?rev=809399&r1=809398&r2=809399&view=diff
==============================================================================
--- 
maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/ModelBuildingRequest.java
 (original)
+++ 
maven/components/trunk/maven-model-builder/src/main/java/org/apache/maven/model/building/ModelBuildingRequest.java
 Sun Aug 30 18:34:46 2009
@@ -269,19 +269,19 @@
     ModelBuildingRequest setModelResolver( ModelResolver modelResolver );
 
     /**
-     * Gets the model building listeners to notify during the build process.
+     * Gets the model building listener to notify during the build process.
      * 
-     * @return The model building listeners to notify, never {...@code null}.
+     * @return The model building listener to notify or {...@code null} if 
none.
      */
-    List<ModelBuildingListener> getModelBuildingListeners();
+    ModelBuildingListener getModelBuildingListener();
 
     /**
-     * Sets the model building listeners to notify during the build process.
+     * Sets the model building listener to notify during the build process.
      * 
-     * @param modelBuildingListeners The model building listeners to notify, 
may be {...@code null}.
+     * @param modelBuildingListener The model building listener to notify, may 
be {...@code null}.
      * @return This request, never {...@code null}.
      */
-    ModelBuildingRequest setModelBuildingListeners( List<? extends 
ModelBuildingListener> modelBuildingListeners );
+    ModelBuildingRequest setModelBuildingListener( ModelBuildingListener 
modelBuildingListener );
 
     /**
      * Gets the model cache to use for reuse of previously built models.


Reply via email to