diff options
| author | Brett Porter <brett@apache.org> | 2006-08-28 07:16:16 +0000 |
|---|---|---|
| committer | Brett Porter <brett@apache.org> | 2006-08-28 07:16:16 +0000 |
| commit | b103fbf4978b76854e5c4c321570c7eb54f09eef (patch) | |
| tree | 1cff515b6209a81828924f89650098bc7805aa42 /archiva-reports-standard | |
| parent | a17f3d69cda3129431439eaaab0fc6eda2246575 (diff) | |
| download | archiva-b103fbf4978b76854e5c4c321570c7eb54f09eef.tar.gz archiva-b103fbf4978b76854e5c4c321570c7eb54f09eef.zip | |
[MRM-144] clarify TODO items
git-svn-id: https://svn.apache.org/repos/asf/maven/archiva/trunk@437583 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'archiva-reports-standard')
3 files changed, 3 insertions, 6 deletions
diff --git a/archiva-reports-standard/src/main/java/org/apache/maven/archiva/reporting/LocationArtifactReportProcessor.java b/archiva-reports-standard/src/main/java/org/apache/maven/archiva/reporting/LocationArtifactReportProcessor.java index e49ef8298..311e6d218 100644 --- a/archiva-reports-standard/src/main/java/org/apache/maven/archiva/reporting/LocationArtifactReportProcessor.java +++ b/archiva-reports-standard/src/main/java/org/apache/maven/archiva/reporting/LocationArtifactReportProcessor.java @@ -152,12 +152,12 @@ public class LocationArtifactReportProcessor } catch ( IOException e ) { - // TODO: should just warn and continue? + // TODO: should just warn and continue! throw new ReportProcessorException( "Unable to read artifact to extract model", e ); } catch ( XmlPullParserException e ) { - // TODO: should just warn and continue? + // TODO: should just warn and continue! throw new ReportProcessorException( "Unable to read artifact to extract model", e ); } finally diff --git a/archiva-reports-standard/src/main/java/org/apache/maven/archiva/reporting/RepositoryQueryLayer.java b/archiva-reports-standard/src/main/java/org/apache/maven/archiva/reporting/RepositoryQueryLayer.java index 19a7956dd..bde683ed9 100644 --- a/archiva-reports-standard/src/main/java/org/apache/maven/archiva/reporting/RepositoryQueryLayer.java +++ b/archiva-reports-standard/src/main/java/org/apache/maven/archiva/reporting/RepositoryQueryLayer.java @@ -31,7 +31,7 @@ public interface RepositoryQueryLayer boolean containsArtifact( Artifact artifact ); /** - * @todo I believe we can remove this [BP] - artifact should contain all the necessary version info + * @todo I believe we can remove this [BP] - artifact should contain all the necessary version info! */ boolean containsArtifact( Artifact artifact, Snapshot snapshot ); diff --git a/archiva-reports-standard/src/test/java/org/apache/maven/archiva/reporting/BadMetadataReportProcessorTest.java b/archiva-reports-standard/src/test/java/org/apache/maven/archiva/reporting/BadMetadataReportProcessorTest.java index 4a189b62a..808773b82 100644 --- a/archiva-reports-standard/src/test/java/org/apache/maven/archiva/reporting/BadMetadataReportProcessorTest.java +++ b/archiva-reports-standard/src/test/java/org/apache/maven/archiva/reporting/BadMetadataReportProcessorTest.java @@ -28,9 +28,6 @@ import org.apache.maven.artifact.repository.metadata.Versioning; import java.util.Iterator; -/** - * @todo??? should use MetadataXpp3Reader instead ? - */ public class BadMetadataReportProcessorTest extends AbstractRepositoryReportsTestCase { |