commit aa05828b76e9f715aa932fea266aaa58fdf8091e
author Michajlo Matijkiw <michajlo@google.com> Mon Sep 28 22:13:27 2015 +0000
committer Florian Weikert <fwe@google.com> Wed Sep 30 09:32:22 2015 +0000
tree c69dcc8ea6a913d6efc3c5002a95cc30de5e9443
parent 1c95806886fc980a011e121352b1fb7c43e9d3cf

Refactor ErrorInfo creation to share single constructor

Single constructor allows us to enforce/document high-level constraints
in a single place. Move previous constructors to static methods which do
their custom transformations but ultimately funnel their final calculated
fields through the common constructor.

Also added some tests to demonstrate expected behavior of static methods.

--
MOS_MIGRATED_REVID=104142909

src/main/java/com/google/devtools/build/skyframe/CycleInfo.java

diff --git a/src/main/java/com/google/devtools/build/skyframe/CycleInfo.java b/src/main/java/com/google/devtools/build/skyframe/CycleInfo.java
index 9759122..a410ec5 100644
--- a/src/main/java/com/google/devtools/build/skyframe/CycleInfo.java
+++ b/src/main/java/com/google/devtools/build/skyframe/CycleInfo.java
@@ -39,7 +39,7 @@
     this(ImmutableList.<SkyKey>of(), cycle);
   }
 
-  CycleInfo(Iterable<SkyKey> pathToCycle, Iterable<SkyKey> cycle) {
+  public CycleInfo(Iterable<SkyKey> pathToCycle, Iterable<SkyKey> cycle) {
     this.pathToCycle = ImmutableList.copyOf(pathToCycle);
     this.cycle = ImmutableList.copyOf(cycle);
   }