commit 8da263cb6275f3822551b4a26544a4b8692199c8
author Nathan Harmata <nharmata@google.com> Tue Feb 24 01:10:22 2015 +0000
committer Han-Wen Nienhuys <hanwen@google.com> Tue Feb 24 01:10:22 2015 +0000
tree 5a79cbd9499be65e01f8b9b4c0e2069eb71ebc37
parent ad5e4cec8862c5e2e2c5e4c055e941f11086860f

Use a factory method in ASTFileLookupValue to enforce that the NO_FILE constant is correctly used.

--
MOS_MIGRATED_REVID=87008436

src/main/java/com/google/devtools/build/lib/skyframe/PackageFunction.java

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/PackageFunction.java b/src/main/java/com/google/devtools/build/lib/skyframe/PackageFunction.java
index c106081..b318f3e 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/PackageFunction.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/PackageFunction.java
@@ -436,7 +436,7 @@
     if (astLookupValue == null) {
       return null;
     }
-    List<Statement> preludeStatements = astLookupValue == ASTFileLookupValue.NO_FILE
+    List<Statement> preludeStatements = astLookupValue.getAST() == null
         ? ImmutableList.<Statement>of() : astLookupValue.getAST().getStatements();
 
     // Load the BUILD file AST and handle Skylark dependencies. This way BUILD files are