diff options
author | cparsons <cparsons@google.com> | 2018-02-12 14:10:17 -0800 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-02-12 14:12:40 -0800 |
commit | b87a47a090aa04b7df256a229ecdbf2ec319c03c (patch) | |
tree | f03da1df72ce4348679152094d56fdaeafed12d5 /src/main/java/com/google/devtools/build/lib/syntax/FuncallExpression.java | |
parent | dd4ddfd4a78e187f3fd39978d569f9b2ae17968b (diff) |
Create a basic annotation processor for validating SkylarkCallable uses at compile time.
RELNOTES: None.
PiperOrigin-RevId: 185432867
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/syntax/FuncallExpression.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/syntax/FuncallExpression.java | 20 |
1 files changed, 0 insertions, 20 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/syntax/FuncallExpression.java b/src/main/java/com/google/devtools/build/lib/syntax/FuncallExpression.java index 241789d268..5584dfdea0 100644 --- a/src/main/java/com/google/devtools/build/lib/syntax/FuncallExpression.java +++ b/src/main/java/com/google/devtools/build/lib/syntax/FuncallExpression.java @@ -101,26 +101,6 @@ public final class FuncallExpression extends Expression { if (callable == null) { continue; } - Preconditions.checkArgument( - callable.parameters().length == 0 || !callable.structField(), - "Method " - + method - + " was annotated with both structField and parameters."); - if (callable.parameters().length > 0 || callable.mandatoryPositionals() >= 0) { - int nbArgs = - callable.parameters().length - + Math.max(0, callable.mandatoryPositionals()); - Preconditions.checkArgument( - nbArgs == method.getParameterTypes().length, - "Method " - + method - + " was annotated for " - + nbArgs - + " arguments " - + "but accept only " - + method.getParameterTypes().length - + " arguments."); - } String name = callable.name(); if (name.isEmpty()) { name = StringUtilities.toPythonStyleFunctionName(method.getName()); |