commit 68c95c56b8891095bad83b7194921c84388ca8a9
author Christian Aistleitner <christian@quelltextlich.at> Sun Apr 14 17:05:39 2013 +0200
committer Christian Aistleitner <christian@quelltextlich.at> Mon Apr 15 01:05:43 2013 +0200
tree 46483292f053e11273fddbc42c0e3f32694dc5b2
parent e2c8126b2e357c895fae838eb6bbbfb61141d815

When searching issue ids in Strings, do not return duplicates

Change-Id: Ia7cfd9589fd12b828d637893177f18918ba7450f

hooks-its/src/main/java/com/googlesource/gerrit/plugins/hooks/util/IssueExtractor.java

diff --git a/hooks-its/src/main/java/com/googlesource/gerrit/plugins/hooks/util/IssueExtractor.java b/hooks-its/src/main/java/com/googlesource/gerrit/plugins/hooks/util/IssueExtractor.java
index c6d4b3d..98d40be 100644
--- a/hooks-its/src/main/java/com/googlesource/gerrit/plugins/hooks/util/IssueExtractor.java
+++ b/hooks-its/src/main/java/com/googlesource/gerrit/plugins/hooks/util/IssueExtractor.java
@@ -1,9 +1,10 @@
 package com.googlesource.gerrit.plugins.hooks.util;
 
-import java.util.ArrayList;
+import java.util.Set;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
+import com.google.common.collect.Sets;
 import com.google.gerrit.server.config.GerritServerConfig;
 import com.google.inject.Inject;
 
@@ -36,7 +37,7 @@
 
     log.debug("Matching '" + haystack + "' against " + pattern.pattern());
 
-    ArrayList<String> issues = new ArrayList<String>();
+    Set<String> issues = Sets.newHashSet();
     Matcher matcher = pattern.matcher(haystack);
 
     while (matcher.find()) {