Merge "Suppress one more java error from Lombok"

1 files changed, 6 insertions, 1 deletions

diff --git a/lint/cli/src/com/android/tools/lint/LombokParser.java b/lint/cli/src/com/android/tools/lint/LombokParser.java
index 0282781..9e96d46 100644
--- a/lint/cli/src/com/android/tools/lint/LombokParser.java
+++ b/lint/cli/src/com/android/tools/lint/LombokParser.java
@@ -17,7 +17,6 @@
 package com.android.tools.lint;
 
 import com.android.tools.lint.client.api.IJavaParser;
-import com.android.tools.lint.client.api.IssueRegistry;
 import com.android.tools.lint.detector.api.JavaContext;
 import com.android.tools.lint.detector.api.Location;
 import com.android.tools.lint.detector.api.Location.Handle;
@@ -48,6 +47,11 @@ public class LombokParser implements IJavaParser {
             // Don't analyze files containing errors
             List<ParseProblem> problems = source.getProblems();
             if (problems != null && problems.size() > 0) {
+                /* Silently ignore the errors. There are still some bugs in Lombok/Parboiled
+                 * (triggered if you run lint on the AOSP framework directory for example),
+                 * and having these show up as fatal errors when it's really a tool bug
+                 * is bad. To make matters worse, the error messages aren't clear:
+                 * http://code.google.com/p/projectlombok/issues/detail?id=313
                 for (ParseProblem problem : problems) {
                     Position position = problem.getPosition();
                     Location location = Location.create(context.file,
@@ -61,6 +65,7 @@ public class LombokParser implements IJavaParser {
                             null);
 
                 }
+                */
                 return null;
             }