Merge pull request #468 from vancem/WarningFix

Fix build warnings

src/PerfView/App.cs

@@ -979,20 +979,22 @@ private static bool CreateConsole()
             });
 
             return true;
-#endif
+#endif 
         }
 
+#if !DOTNET_CORE
         static Thread s_threadToInterrupt;
         static int s_controlCPressed = 0;
+#endif
 
-        #endregion
-        #endregion
-    }
+#endregion
+#endregion
+        }
 
-    /// <summary>
-    /// APIs for logging usage data and feedback.
-    /// </summary>
-    public static class AppLog
+        /// <summary>
+        /// APIs for logging usage data and feedback.
+        /// </summary>
+        public static class AppLog
     {
         /// <summary>
         /// Returns true if you have access to the file share where we log feedback
@@ -1146,7 +1148,7 @@ public static string BuildDate
             }
         }
 
-        #region private
+#region private
 
 
         private static string FeedbackServer { get { return "clrMain"; } }
@@ -1189,7 +1191,7 @@ private static bool WriteFeedbackToLog(string filePath, string message)
             }
             return false;
         }
-        #endregion
+#endregion
     }
 
     /// <summary>
@@ -1235,9 +1237,9 @@ protected override void Dispose(bool disposing)
             m_terseLog.Dispose();
             m_verboseLog.Dispose();
         }
-        #region private
+#region private
         TextWriter m_verboseLog;
         TextWriter m_terseLog;
-        #endregion
+#endregion
     }
 }

src/TraceEvent/Stacks/RecursionGuard.cs

@@ -3,8 +3,7 @@
 namespace Microsoft.Diagnostics.Tracing.Stacks
 {
     /// <summary>
-    /// This structure provides a clean API for a lightweight recursion stack guard to prevent
-    /// <see cref="StackOverflowException"/>.
+    /// This structure provides a clean API for a lightweight recursion stack guard to prevent StackOverflow exceptions
     /// </summary>
     internal struct RecursionGuard
     {