From 88d1caeef07533429468515f63b3d4e2cb9a7a80 Mon Sep 17 00:00:00 2001 From: cparsons Date: Fri, 22 Jun 2018 15:12:00 -0700 Subject: Make @SkylarkCallable.name mandatory. RELNOTES: None. PiperOrigin-RevId: 201748802 --- .../com/google/devtools/build/lib/syntax/SkylarkSignatureProcessor.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/main/java/com/google/devtools/build/lib/syntax/SkylarkSignatureProcessor.java') diff --git a/src/main/java/com/google/devtools/build/lib/syntax/SkylarkSignatureProcessor.java b/src/main/java/com/google/devtools/build/lib/syntax/SkylarkSignatureProcessor.java index 5d5094117d..f5311328f4 100644 --- a/src/main/java/com/google/devtools/build/lib/syntax/SkylarkSignatureProcessor.java +++ b/src/main/java/com/google/devtools/build/lib/syntax/SkylarkSignatureProcessor.java @@ -51,7 +51,7 @@ public class SkylarkSignatureProcessor { SkylarkCallable annotation = descriptor.getAnnotation(); // TODO(cparsons): Validate these properties with the annotation processor instead. - Preconditions.checkArgument(annotation.name().isEmpty() || name.equals(annotation.name()), + Preconditions.checkArgument(name.equals(annotation.name()), "%s != %s", name, annotation.name()); boolean documented = annotation.documented(); if (annotation.doc().isEmpty() && documented) { -- cgit v1.2.3