commit 6e5eecb1d7a25f8f38a2aed681621be91db7fd0f
author Laurent Le Brun <laurentlb@google.com> Thu Sep 10 11:37:32 2015 +0000
committer Damien Martin-Guillerez <dmarting@google.com> Fri Sep 11 09:45:17 2015 +0000
tree 26cac04c22a8c4ddf788af5ba9517d81c7d29112
parent 4aa2912d12dbcede9bf74397a4269499fc153fd4

Preserve the original location in EvalExceptionWithStackTrace

--
MOS_MIGRATED_REVID=102744198

src/main/java/com/google/devtools/build/lib/syntax/EvalExceptionWithStackTrace.java

diff --git a/src/main/java/com/google/devtools/build/lib/syntax/EvalExceptionWithStackTrace.java b/src/main/java/com/google/devtools/build/lib/syntax/EvalExceptionWithStackTrace.java
index 82ca835..8688465 100644
--- a/src/main/java/com/google/devtools/build/lib/syntax/EvalExceptionWithStackTrace.java
+++ b/src/main/java/com/google/devtools/build/lib/syntax/EvalExceptionWithStackTrace.java
@@ -14,6 +14,7 @@
 package com.google.devtools.build.lib.syntax;
 
 import com.google.common.base.Joiner;
+import com.google.common.base.MoreObjects;
 import com.google.common.base.Preconditions;
 import com.google.devtools.build.lib.events.Location;
 import com.google.devtools.build.lib.packages.Rule;
@@ -30,7 +31,19 @@
   private StackTraceElement mostRecentElement;
 
   public EvalExceptionWithStackTrace(Exception original, Location callLocation) {
-    super(callLocation, getNonEmptyMessage(original), getCause(original));
+    super(
+        originalLocation(original, callLocation), getNonEmptyMessage(original), getCause(original));
+  }
+
+  /**
+   * Returns the location of the {@code original} exception, or {@code callLocation}
+   * if there's none.
+   */
+  private static Location originalLocation(Exception original, Location callLocation) {
+    if (!(original instanceof EvalException)) {
+      return callLocation;
+    }
+    return MoreObjects.firstNonNull(((EvalException) original).getLocation(), callLocation);
   }
 
   /**