An empty profiler report, which just a header, is possible

smera · Dec 5, 2017 · ffb51d8 · ffb51d8
1 parent 25a7c09
commit ffb51d8
Showing 1 changed file with 12 additions and 5 deletions.
diff --git a/src/Build/Evaluation/Profiler/ProfilerResultPrettyPrinter.cs b/src/Build/Evaluation/Profiler/ProfilerResultPrettyPrinter.cs
@@ -27,18 +27,25 @@ public sealed class ProfilerResultPrettyPrinter
         /// </summary>
         public static string GetMarkdownContent(ProfilerResult result)
         {
+            var stringBuilder = new StringBuilder();
+
+            stringBuilder.AppendLine("Pass|File|Line #|Expression|Inc (ms)|Inc (%)|Exc (ms)|Exc (%)|#|Bug");
+            stringBuilder.AppendLine("---|---|---:|---|---:|---:|---:|---:|---:|---");
+
             var profiledLocations = result.ProfiledLocations;
+
+            // If there are no profiled locations, then just return
+            if (profiledLocations.Count == 0)
+            {
+                return stringBuilder.ToString();
+            }
+
             var evaluationPasses = profiledLocations.Where(l => l.Key.File == null)
                                                   .OrderBy(l => l.Key.EvaluationPass);
 
             var orderedLocations = profiledLocations.Where(l => l.Key.File != null)
                                                   .OrderByDescending(l => l.Value.ExclusiveTime);
 
-            var stringBuilder = new StringBuilder();
-
-            stringBuilder.AppendLine("Pass|File|Line #|Expression|Inc (ms)|Inc (%)|Exc (ms)|Exc (%)|#|Bug");
-            stringBuilder.AppendLine("---|---|---:|---|---:|---:|---:|---:|---:|---");
-
             TimeSpan? totalTime = null;
             foreach (var pair in evaluationPasses)
             {