From c34ee5bc2208fc8977f033e54de83a78673580b5 Mon Sep 17 00:00:00 2001
From: Elliotte Rusty Harold <elharo@users.noreply.github.com>
Date: Sun, 2 Jul 2023 12:01:51 +0000
Subject: [PATCH 1/3] Rewrite API doc

Noticed this was a unclear and out of date
---
 .../org/apache/maven/artifact/handler/ArtifactHandler.java | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java b/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java
index 51c7c51d2c9a..2e7c80963644 100644
--- a/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java
+++ b/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java
@@ -19,7 +19,7 @@
 package org.apache.maven.artifact.handler;
 
 /**
- * An artifact handler defines for a dependency type, defined as Plexus role:<ul>
+ * An artifact handler provides metadata for an artifact derived from the dependency element that references the artifact:<ul>
  * <li>extension and classifier, to be able to download the file,</li>
  * <li>information on how to use the artifact: whether to add it to the classpath, or to take into account its
  * dependencies.</li>
@@ -32,7 +32,8 @@ public interface ArtifactHandler {
     String ROLE = ArtifactHandler.class.getName();
 
     /**
-     * Get the file extension associated to the file represented by the dependency type.
+     * Returns the file name extension used for dependencies of that type;
+     * e.g. "jar", "pom", "xml", etc.
      *
      * @return the file extension
      */
@@ -41,7 +42,7 @@ public interface ArtifactHandler {
     String getDirectory();
 
     /**
-     * Get the classifier associated to the dependency type.
+     * Returns the classifier of the dependency.
      *
      * @return the classifier
      */

From 9c82350d8ed53c447ee839a1f2e523f0bec83aa2 Mon Sep 17 00:00:00 2001
From: Elliotte Rusty Harold <elharo@ibiblio.org>
Date: Tue, 4 Jul 2023 08:28:48 -0400
Subject: [PATCH 2/3] correct text

---
 .../apache/maven/artifact/handler/ArtifactHandler.java    | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java b/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java
index 2e7c80963644..0b4e143788ea 100644
--- a/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java
+++ b/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java
@@ -19,10 +19,10 @@
 package org.apache.maven.artifact.handler;
 
 /**
- * An artifact handler provides metadata for an artifact derived from the dependency element that references the artifact:<ul>
- * <li>extension and classifier, to be able to download the file,</li>
+ * An artifact handler contains metadata derived from the dependency element that references the artifact:<ul>
+ * <li>extension and classifier, to be able to download the file</li>
  * <li>information on how to use the artifact: whether to add it to the classpath, or to take into account its
- * dependencies.</li>
+ * dependencies</li>
  * </ul>
  *
  * @author <a href="mailto:jason@maven.org">Jason van Zyl</a>
@@ -32,7 +32,7 @@ public interface ArtifactHandler {
     String ROLE = ArtifactHandler.class.getName();
 
     /**
-     * Returns the file name extension used for dependencies of that type;
+     * Returns the file name extension of the artifact;
      * e.g. "jar", "pom", "xml", etc.
      *
      * @return the file extension

From c0bbc96297515759f8e2d39120dd0442d864da97 Mon Sep 17 00:00:00 2001
From: Elliotte Rusty Harold <elharo@ibiblio.org>
Date: Fri, 14 Jul 2023 07:59:28 -0400
Subject: [PATCH 3/3] revise

---
 .../apache/maven/artifact/handler/ArtifactHandler.java | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java b/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java
index 0b4e143788ea..89f59bdb284e 100644
--- a/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java
+++ b/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java
@@ -19,10 +19,10 @@
 package org.apache.maven.artifact.handler;
 
 /**
- * An artifact handler contains metadata derived from the dependency element that references the artifact:<ul>
- * <li>extension and classifier, to be able to download the file</li>
- * <li>information on how to use the artifact: whether to add it to the classpath, or to take into account its
- * dependencies</li>
+ * An artifact handler contains information explaining how an artifact plugs into the Maven build:<ul>
+ * <li>Information needed to find the artifact file in a repository including extension and classifier</li>
+ * <li>Information on how to use the artifact as a dependency: whether to add it to the classpath, whether to load its
+ * dependencies transitively</li>
  * </ul>
  *
  * @author <a href="mailto:jason@maven.org">Jason van Zyl</a>
@@ -42,7 +42,7 @@ public interface ArtifactHandler {
     String getDirectory();
 
     /**
-     * Returns the classifier of the dependency.
+     * Returns the default classifier used if a different one is not set in pom.xml.
      *
      * @return the classifier
      */