From cbd81d7e9cd3bbd3545af8e7285b660ad4bc5de7 Mon Sep 17 00:00:00 2001
From: kaz <kaz@13f79535-47bb-0310-9956-ffa450edef68>
Date: Fri, 31 Jan 2003 18:58:49 +0000
Subject: [PATCH] Follow up to my last commit.  The HashSet wasn't preserving
 order, so we should use a LinkedHashSet, but that isn't part of 1.3.  So
 we'll just use an ArrayList and create a new list each time this goal is
 invoked which will prevent duplicates from being added.

git-svn-id: https://svn.apache.org/repos/asf/maven/maven-1/plugins/trunk@112790 13f79535-47bb-0310-9956-ffa450edef68
---
 xdoc/plugin.jelly | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/xdoc/plugin.jelly b/xdoc/plugin.jelly
index 18e5c9e7..15ee901d 100644
--- a/xdoc/plugin.jelly
+++ b/xdoc/plugin.jelly
@@ -431,7 +431,17 @@
 
   <goal name="xdoc:register-reports">
 
-    <j:new var="reports" className="java.util.HashSet"/>
+    <!--
+     | This should be a LinkedHashSet to prevent someone from
+     | registering somthing twice.  Unfortunately, 1.3 does not 
+     | have a LinkedHashSet so I'll leave this here as a reminder
+     | for us someday.  In the meantime, each time this goal is called,
+     | we'll just replace the old List with a new one.  Shouldn't cause
+     | any problmes.  When we do actually use the Set, we can put a check
+     | to see if 'reports' already exists, and if so, do nothing.
+     | <j:new var="reports" className="java.util.LinkedHashSet"/>
+     |-->
+    <j:new var="reports" className="java.util.ArrayList"/>
     <j:set var="dummy" value="${pom.getPluginContext('maven-xdoc-plugin').setVariable('reports', reports)}"/>
 
     <!--