Change hasChildChanged method name to childCausesReevaluation, to indicate that we may force the parent to re-evaluate even if the child's value hasn't changed.
PiperOrigin-RevId: 215644644
diff --git a/src/main/java/com/google/devtools/build/skyframe/InMemoryNodeEntry.java b/src/main/java/com/google/devtools/build/skyframe/InMemoryNodeEntry.java
index d498c41..fcc1d3b 100644
--- a/src/main/java/com/google/devtools/build/skyframe/InMemoryNodeEntry.java
+++ b/src/main/java/com/google/devtools/build/skyframe/InMemoryNodeEntry.java
@@ -463,7 +463,7 @@
signaledDeps++;
if (isDirty()) {
dirtyBuildingState.signalDepInternal(
- hasChildChanged(lastEvaluatedVersion, childVersion), isReady());
+ childCausesReevaluation(lastEvaluatedVersion, childVersion), isReady());
}
return isReady();
}
@@ -654,8 +654,8 @@
return isReady(getNumTemporaryDirectDeps());
}
- /** True if the child has changed since the last evaluated version. */
- protected boolean hasChildChanged(Version lastEvaluatedVersion, Version childVersion) {
+ /** True if the child should cause re-evaluation of this node. */
+ protected boolean childCausesReevaluation(Version lastEvaluatedVersion, Version childVersion) {
// childVersion > lastEvaluatedVersion
return !childVersion.atMost(lastEvaluatedVersion);
}