Skip to content
Snippets Groups Projects
Commit 2beb1144 authored by Cole Faust's avatar Cole Faust Committed by Automerger Merge Worker
Browse files

Merge "Fixes for errorprone update" into main am: 67bb3d4e

parents 45c0b3fe 67bb3d4e
No related branches found
No related tags found
No related merge requests found
...@@ -66,6 +66,7 @@ android_test { ...@@ -66,6 +66,7 @@ android_test {
errorprone: { errorprone: {
javacflags: [ javacflags: [
"-Xep:ReturnValueIgnored:WARN", "-Xep:ReturnValueIgnored:WARN",
"-Xep:UnnecessaryStringBuilder:OFF",
], ],
}, },
} }
...@@ -63,7 +63,7 @@ public class HideInCommentsChecker extends BugChecker implements ...@@ -63,7 +63,7 @@ public class HideInCommentsChecker extends BugChecker implements
@Override @Override
public Description matchCompilationUnit(CompilationUnitTree tree, VisitorState state) { public Description matchCompilationUnit(CompilationUnitTree tree, VisitorState state) {
final Map<Integer, Tree> javadocableTrees = findJavadocableTrees(tree); final Map<Integer, Tree> javadocableTrees = findJavadocableTrees(tree, state);
final String sourceCode = state.getSourceCode().toString(); final String sourceCode = state.getSourceCode().toString();
for (ErrorProneToken token : ErrorProneTokens.getTokens(sourceCode, state.context)) { for (ErrorProneToken token : ErrorProneTokens.getTokens(sourceCode, state.context)) {
for (Tokens.Comment comment : token.comments()) { for (Tokens.Comment comment : token.comments()) {
...@@ -112,9 +112,9 @@ public class HideInCommentsChecker extends BugChecker implements ...@@ -112,9 +112,9 @@ public class HideInCommentsChecker extends BugChecker implements
} }
private Map<Integer, Tree> findJavadocableTrees(CompilationUnitTree tree) { private Map<Integer, Tree> findJavadocableTrees(CompilationUnitTree tree, VisitorState state) {
Map<Integer, Tree> javadoccableTrees = new HashMap<>(); Map<Integer, Tree> javadoccableTrees = new HashMap<>();
new SuppressibleTreePathScanner<Void, Void>() { new SuppressibleTreePathScanner<Void, Void>(state) {
@Override @Override
public Void visitClass(ClassTree classTree, Void unused) { public Void visitClass(ClassTree classTree, Void unused) {
javadoccableTrees.put(getStartPosition(classTree), classTree); javadoccableTrees.put(getStartPosition(classTree), classTree);
......
...@@ -212,6 +212,11 @@ android_library { ...@@ -212,6 +212,11 @@ android_library {
lint: { lint: {
extra_check_modules: ["SystemUILintChecker"], extra_check_modules: ["SystemUILintChecker"],
}, },
errorprone: {
javacflags: [
"-Xep:InvalidPatternSyntax:WARN",
],
},
} }
filegroup { filegroup {
...@@ -427,6 +432,11 @@ android_library { ...@@ -427,6 +432,11 @@ android_library {
test: true, test: true,
extra_check_modules: ["SystemUILintChecker"], extra_check_modules: ["SystemUILintChecker"],
}, },
errorprone: {
javacflags: [
"-Xep:InvalidPatternSyntax:WARN",
],
},
} }
android_app { android_app {
...@@ -464,6 +474,12 @@ android_app { ...@@ -464,6 +474,12 @@ android_app {
}, },
plugins: ["dagger2-compiler"], plugins: ["dagger2-compiler"],
errorprone: {
javacflags: [
"-Xep:InvalidPatternSyntax:WARN",
],
},
} }
android_robolectric_test { android_robolectric_test {
......
...@@ -11,4 +11,7 @@ android_test { ...@@ -11,4 +11,7 @@ android_test {
name: "SmokeTestTriggerApps", name: "SmokeTestTriggerApps",
srcs: ["src/**/*.java"], srcs: ["src/**/*.java"],
sdk_version: "current", sdk_version: "current",
errorprone: {
enabled: false,
},
} }
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment