Re: svn commit: r330080 - in /maven/components/trunk: maven-artifact/src/main/java/org/apache/maven/artifact/factory/ maven-artifact/src/main/java/org/apache/maven/artifact/versioning/ maven-project/src/main/java/org/apache/maven/project/

[Brett Porter]

[EMAIL PROTECTED] wrote:
+        this.artifact = new DefaultArtifactFactory().cloneArtifact( 
project.artifact );

This doesn't seem very component-oriented :)

Maybe clone should be a method of artifact (not using the standard Java 
one, but createCopy() or something). The clone method doesn't seem to 
have any relationship to the factory.

- Brett

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]