Fix source/line info in unhandled exceptions (#5874)
authorMike McLaughlin <mikem@microsoft.com>
Sun, 19 Jun 2016 22:24:49 +0000 (15:24 -0700)
committerGitHub <noreply@github.com>
Sun, 19 Jun 2016 22:24:49 +0000 (15:24 -0700)
* Fix no source/line info for unhandled exceptions.

Issue #5828.

* Remove comment.

src/mscorlib/src/System/Exception.cs

index 51127a513c32770f2598b9e328fb0e835b89c501..b08c12de614c78867afee0c4b041bf9f4464c467 100644 (file)
@@ -910,19 +910,8 @@ namespace System {
                 //however if something wrong happens we still can call the usual ToString
             }
 
-            // Get the current stack trace string.  On CoreCLR we don't bother
-            // to try and include file/line-number information because all AppDomains
-            // are sandboxed, and so this won't succeed in most (or all) cases.  Therefore the
-            // Demand and exception overhead is a waste.
-            // We currently have some bugs in watson bucket generation where the SecurityException
-            // here causes us to lose saved bucket parameters.  By not even doing the demand
-            // we avoid those problems (although there are deep underlying problems that need to
-            // be fixed there - relying on this to avoid problems is incomplete and brittle).
-            bool fGetFileLineInfo = true;
-#if FEATURE_CORECLR
-            fGetFileLineInfo = false;
-#endif
-            return ToString(fGetFileLineInfo, true);
+            // Get the current stack trace string. 
+            return ToString(true, true);
         }
 
 #if !FEATURE_CORECLR