commit | ab3cdb7eabcdedebf1cb50e2e1b3d02d2280ec49 | [log] [tgz] |
---|---|---|
author | Googler <noreply@google.com> | Thu Nov 14 14:11:24 2019 -0800 |
committer | Copybara-Service <copybara-worker@google.com> | Thu Nov 14 14:12:22 2019 -0800 |
tree | 32abe64fa0d912aa5a67ddd9527b6ec0115ff0f1 | |
parent | a6ffdf9274788d18d830cdf598fefc66e17b5119 [diff] |
Make SkylarkCallableProcessor#process return false Returning true consumes the annotation and hides it from other post processors. Returning false allows other processors to also act upon the annotations. PiperOrigin-RevId: 280509697
diff --git a/src/main/java/com/google/devtools/build/lib/skylarkinterface/processor/SkylarkCallableProcessor.java b/src/main/java/com/google/devtools/build/lib/skylarkinterface/processor/SkylarkCallableProcessor.java index 8ff22e2..d7930e3 100644 --- a/src/main/java/com/google/devtools/build/lib/skylarkinterface/processor/SkylarkCallableProcessor.java +++ b/src/main/java/com/google/devtools/build/lib/skylarkinterface/processor/SkylarkCallableProcessor.java
@@ -189,7 +189,8 @@ } } - return true; + // Returning false allows downstream processors to work on the same annotations + return false; } private void verifyNoNameConflict(ExecutableElement methodElement, SkylarkCallable annotation)