svn commit: r1372382 - in /maven/indexer/trunk/indexer-artifact/src: main/java/org/apache/maven/index/artifact/ test/java/org/apache/maven/index/artifact/

[cs...@apache.org]

Author: cstamas
Date: Mon Aug 13 12:09:42 2012
New Revision: 1372382

URL: http://svn.apache.org/viewvc?rev=1372382&view=rev
Log:
Getting rid of deprecated baggage.

Removed:
    maven/indexer/trunk/indexer-artifact/src/main/java/org/apache/maven/index/artifact/IllegalArtifactCoordinateException.java
Modified:
    maven/indexer/trunk/indexer-artifact/src/main/java/org/apache/maven/index/artifact/Gav.java
    maven/indexer/trunk/indexer-artifact/src/main/java/org/apache/maven/index/artifact/GavCalculator.java
    maven/indexer/trunk/indexer-artifact/src/test/java/org/apache/maven/index/artifact/M2GavCalculatorTest.java

Modified: maven/indexer/trunk/indexer-artifact/src/main/java/org/apache/maven/index/artifact/Gav.java
URL: http://svn.apache.org/viewvc/maven/indexer/trunk/indexer-artifact/src/main/java/org/apache/maven/index/artifact/Gav.java?rev=1372382&r1=1372381&r2=1372382&view=diff
==============================================================================
--- maven/indexer/trunk/indexer-artifact/src/main/java/org/apache/maven/index/artifact/Gav.java (original)
+++ maven/indexer/trunk/indexer-artifact/src/main/java/org/apache/maven/index/artifact/Gav.java Mon Aug 13 12:09:42 2012
@@ -93,33 +93,6 @@ public class Gav
         this( groupId, artifactId, version, null, null, null, null, null, false, null, false, null );
     }
 
-    /**
-     * Deprecated constructor, left here for backward compatibility. It simply delegates to other constructor and
-     * neglects the snapshot redundant parameter.
-     * 
-     * @deprecated The <code>boolean snapshot</code> parameter is simply neglected. Use the constructor without it.
-     * @param groupId
-     * @param artifactId
-     * @param version
-     * @param classifier
-     * @param extension
-     * @param snapshotBuildNumber
-     * @param snapshotTimeStamp
-     * @param name
-     * @param snapshot
-     * @param hash
-     * @param hashType
-     * @param signature
-     * @param signatureType
-     */
-    public Gav( String groupId, String artifactId, String version, String classifier, String extension,
-                Integer snapshotBuildNumber, Long snapshotTimeStamp, String name, boolean snapshot, boolean hash,
-                HashType hashType, boolean signature, SignatureType signatureType )
-    {
-        this( groupId, artifactId, version, classifier, extension, snapshotBuildNumber, snapshotTimeStamp, name, hash,
-            hashType, signature, signatureType );
-    }
-
     public Gav( String groupId, String artifactId, String version, String classifier, String extension,
                 Integer snapshotBuildNumber, Long snapshotTimeStamp, String name, boolean hash, HashType hashType,
                 boolean signature, SignatureType signatureType )

Modified: maven/indexer/trunk/indexer-artifact/src/main/java/org/apache/maven/index/artifact/GavCalculator.java
URL: http://svn.apache.org/viewvc/maven/indexer/trunk/indexer-artifact/src/main/java/org/apache/maven/index/artifact/GavCalculator.java?rev=1372382&r1=1372381&r2=1372382&view=diff
==============================================================================
--- maven/indexer/trunk/indexer-artifact/src/main/java/org/apache/maven/index/artifact/GavCalculator.java (original)
+++ maven/indexer/trunk/indexer-artifact/src/main/java/org/apache/maven/index/artifact/GavCalculator.java Mon Aug 13 12:09:42 2012
@@ -31,14 +31,12 @@ public interface GavCalculator
      * Calculates GAV from provided <em>repository path</em>. The path has to be absolute starting from repository root.
      * If path represents a proper artifact path (conforming to given layout), GAV is "calculated" from it and is
      * returned. If path represents some file that is not an artifact, but is part of the repository layout (like
-     * maven-metadata.xml), or in any other case it returns null. TODO: some place for different levels of "validation"?
+     * maven-metadata.xml), or in any other case it returns null.
      * 
      * @param path the repository path
      * @return Gav parsed from the path
-     * @throws IllegalArtifactCoordinateException not throwing it anymore, here just for backward compatibility.
      */
-    Gav pathToGav( String path )
-        throws IllegalArtifactCoordinateException;
+    Gav pathToGav( String path );
 
     /**
      * Reassembles the repository path from the supplied GAV. It will be an absolute path.

Modified: maven/indexer/trunk/indexer-artifact/src/test/java/org/apache/maven/index/artifact/M2GavCalculatorTest.java
URL: http://svn.apache.org/viewvc/maven/indexer/trunk/indexer-artifact/src/test/java/org/apache/maven/index/artifact/M2GavCalculatorTest.java?rev=1372382&r1=1372381&r2=1372382&view=diff
==============================================================================
--- maven/indexer/trunk/indexer-artifact/src/test/java/org/apache/maven/index/artifact/M2GavCalculatorTest.java (original)
+++ maven/indexer/trunk/indexer-artifact/src/test/java/org/apache/maven/index/artifact/M2GavCalculatorTest.java Mon Aug 13 12:09:42 2012
@@ -571,7 +571,6 @@ public class M2GavCalculatorTest
         throws Exception
     {
         Gav gav;
-        String path;
 
         // NEXUS-4132
         gav =