From 21495dd2c03b182908a676337e8fada072923b37 Mon Sep 17 00:00:00 2001
From: WonderCsabo <kozakcsabi@gmail.com>
Date: Tue, 9 Sep 2014 21:10:41 +0200
Subject: [PATCH] Fix processor option format with maven-compiler compilerArgs

---
 .../maven/apt/internal/utils/ProjectUtils.java    | 15 +++++----------
 .../projects/compilerArgs/pom.xml                 |  4 ++--
 .../tools/maven/apt/tests/ProjectUtilsTest.java   |  2 +-
 3 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/org.jboss.tools.maven.apt.core/src/org/jboss/tools/maven/apt/internal/utils/ProjectUtils.java b/org.jboss.tools.maven.apt.core/src/org/jboss/tools/maven/apt/internal/utils/ProjectUtils.java
index 4b15003..2981256 100644
--- a/org.jboss.tools.maven.apt.core/src/org/jboss/tools/maven/apt/internal/utils/ProjectUtils.java
+++ b/org.jboss.tools.maven.apt.core/src/org/jboss/tools/maven/apt/internal/utils/ProjectUtils.java
@@ -42,22 +42,19 @@
  */
 public class ProjectUtils {
 
+  private static final Pattern OPTION_PATTERN = Pattern.compile("-A([^ \\t\"']+)");
+  
   /**
    * Parse a string to extract Annotation Processor options
    */
   public static Map<String, String> parseProcessorOptions(String compilerArgument) {
-    Pattern fullOptionPattern = Pattern.compile("-A([^ \\t\"']+)");
-    return parseProcessorOptions(compilerArgument, fullOptionPattern);
-  }
-  
-  private static Map<String, String> parseProcessorOptions(String compilerArgument, Pattern pattern) {
       
     if (compilerArgument == null || compilerArgument.trim().isEmpty()) {
       return Collections.emptyMap();
     }
     Map<String, String> ret = new HashMap<String, String>();
     
-    Matcher matcher = pattern.matcher(compilerArgument);
+    Matcher matcher = OPTION_PATTERN.matcher(compilerArgument);
     
     int start = 0;
     while(matcher.find(start)) {
@@ -90,11 +87,9 @@ public static Map<String, String> parseProcessorOptions(List<String> compilerArg
     }
     Map<String, String> options = new HashMap<String, String>();
     
-    Pattern pattern = Pattern.compile("A([^ \\t\"']+)");
-    
     for (String arg : compilerArgs) {
-      if (arg.startsWith("A")) {
-        options.putAll(parseProcessorOptions(arg, pattern));
+      if (arg.startsWith("-A")) {
+        options.putAll(parseProcessorOptions(arg));
       }
     }
     return options;
diff --git a/org.jboss.tools.maven.apt.tests/projects/compilerArgs/pom.xml b/org.jboss.tools.maven.apt.tests/projects/compilerArgs/pom.xml
index 9962205..b0df337 100644
--- a/org.jboss.tools.maven.apt.tests/projects/compilerArgs/pom.xml
+++ b/org.jboss.tools.maven.apt.tests/projects/compilerArgs/pom.xml
@@ -18,8 +18,8 @@
 						<AaddGenerationDate>true</AaddGenerationDate>
 					</compilerArguments>
                     <compilerArgs>
-                        <arg>AaddGeneratedAnnotation=true</arg>
-                        <arg>AcompilerArg</arg>
+                        <arg>-AaddGeneratedAnnotation=true</arg>
+                        <arg>-AcompilerArg</arg>
                     </compilerArgs>
 				</configuration>
 			</plugin>
diff --git a/org.jboss.tools.maven.apt.tests/src/org/jboss/tools/maven/apt/tests/ProjectUtilsTest.java b/org.jboss.tools.maven.apt.tests/src/org/jboss/tools/maven/apt/tests/ProjectUtilsTest.java
index 1d6704a..22fd478 100644
--- a/org.jboss.tools.maven.apt.tests/src/org/jboss/tools/maven/apt/tests/ProjectUtilsTest.java
+++ b/org.jboss.tools.maven.apt.tests/src/org/jboss/tools/maven/apt/tests/ProjectUtilsTest.java
@@ -30,7 +30,7 @@ public class ProjectUtilsTest {
 
   @Test
   public void testParseCompilerArgs() {
-    List<String> compilerArgs = Arrays.asList("Afoo=bar","Abracadabra", "Xman");
+    List<String> compilerArgs = Arrays.asList("-Afoo=bar","-Abracadabra", "Xman");
     
     Map<String, String> result = ProjectUtils.parseProcessorOptions(compilerArgs);
     assertNotNull(result);