commit cf738441e9cdda0f9d08861cb316596f77821baa
author Nathan Harmata <nharmata@google.com> Tue Jun 28 14:52:21 2016 +0000
committer Dmitry Lomov <dslomov@google.com> Wed Jun 29 08:53:28 2016 +0000
tree 79eea6df358c35601df7e423d0e15de94519e509
parent c079541bcda457ccbb82eacdae54ddb5d2ddc324

Expose versions of Package.Builder#addRule and RuleClass#createRule that don't perform internal sanity checks.

--
MOS_MIGRATED_REVID=126073693

src/main/java/com/google/devtools/build/lib/packages/RuleClass.java

diff --git a/src/main/java/com/google/devtools/build/lib/packages/RuleClass.java b/src/main/java/com/google/devtools/build/lib/packages/RuleClass.java
index 6f4ffd7..a9ef960 100644
--- a/src/main/java/com/google/devtools/build/lib/packages/RuleClass.java
+++ b/src/main/java/com/google/devtools/build/lib/packages/RuleClass.java
@@ -1340,6 +1340,26 @@
       Location location,
       AttributeContainer attributeContainer)
       throws LabelSyntaxException, InterruptedException {
+    Rule rule = createRuleUnchecked(
+        pkgBuilder, ruleLabel, attributeValues, eventHandler, ast, location, attributeContainer);
+    rule.checkForNullLabels();
+    return rule;
+  }
+
+  /**
+   * Same as {@link #createRule}, except without some internal sanity checks.
+   *
+   * <p>Don't call this function unless you know what you're doing.
+   */
+  Rule createRuleUnchecked(
+      Package.Builder pkgBuilder,
+      Label ruleLabel,
+      AttributeValuesMap attributeValues,
+      EventHandler eventHandler,
+      @Nullable FuncallExpression ast,
+      Location location,
+      AttributeContainer attributeContainer)
+      throws LabelSyntaxException, InterruptedException {
     Rule rule = pkgBuilder.createRule(ruleLabel, this, location, attributeContainer);
     populateRuleAttributeValues(rule, pkgBuilder, attributeValues, eventHandler);
     checkAspectAllowedValues(rule, eventHandler);
@@ -1350,7 +1370,6 @@
     checkForDuplicateLabels(rule, eventHandler);
     checkThirdPartyRuleHasLicense(rule, pkgBuilder, eventHandler);
     checkForValidSizeAndTimeoutValues(rule, eventHandler);
-    rule.checkForNullLabels();
     rule.checkValidityPredicate(eventHandler);
     return rule;
   }