Added a verbose switch

There really isn't a reason to print out the times for every file in a
default run.

Author: jaredpar

src/CodeFormatter/Program.cs

@@ -24,13 +24,14 @@ private static int Main(string[] args)
         {
             if (args.Length < 1)
             {
-                Console.WriteLine("CodeFormatter <project or solution> [/file:<filename>] [/nocopyright] [/nounicode] [/tables] [/c:<config1,config2> [/copyright:file]");
+                Console.WriteLine("CodeFormatter <project or solution> [/file:<filename>] [/nocopyright] [/nounicode] [/tables] [/c:<config1,config2> [/copyright:file] [/verbose]");
                 Console.WriteLine("    <filename>   - Only apply changes to files with specified name.");
                 Console.WriteLine("    <configs>    - Additional preprocessor configurations the formatter");
                 Console.WriteLine("                   should run under.");
                 Console.WriteLine("    <copyright>  - Specifies file containing copyright header.");
                 Console.WriteLine("                   Use ConvertTests to convert MSTest tests to xUnit.");
                 Console.WriteLine("    <tables>     - Let tables opt out of formatting by defining DOTNET_FORMATTER");
+                Console.WriteLine("    <verbose>    - Verbose output");
                 Console.WriteLine("    <nounicode>  - Do not convert unicode strings to escape sequences");
                 Console.WriteLine("    <nocopyright>- Do not update the copyright message.");
                 return -1;
@@ -49,6 +50,7 @@ private static int Main(string[] args)
             var copyrightHeader = FormattingConstants.DefaultCopyrightHeader;
             var convertUnicode = true;
             var allowTables = false;
+            var verbose = false;
             var comparer = StringComparer.OrdinalIgnoreCase;
 
             for (int i = 1; i < args.Length; i++)
@@ -88,6 +90,10 @@ private static int Main(string[] args)
                 {
                     convertUnicode = false;
                 }
+                else if (comparer.Equals(arg, "/verbose"))
+                {
+                    verbose = true;
+                }
                 else if (comparer.Equals(arg, "/tables"))
                 {
                     allowTables = true;
@@ -113,6 +119,7 @@ private static int Main(string[] args)
                     copyrightHeader,
                     allowTables,
                     convertUnicode,
+                    verbose,
                     ct).Wait(ct);
                 Console.WriteLine("Completed formatting.");
                 return 0;
@@ -140,6 +147,7 @@ private static async Task RunAsync(
             ImmutableArray<string> copyrightHeader,
             bool allowTables,
             bool convertUnicode,
+            bool verbose,
             CancellationToken cancellationToken)
         {
             var workspace = MSBuildWorkspace.Create();
@@ -149,7 +157,9 @@ private static async Task RunAsync(
             engine.CopyrightHeader = copyrightHeader;
             engine.AllowTables = allowTables;
             engine.ConvertUnicodeCharacters = convertUnicode;
+            engine.Verbose = verbose;
 
+            Console.WriteLine(Path.GetFileName(projectOrSolutionPath));
             string extension = Path.GetExtension(projectOrSolutionPath);
             if (StringComparer.OrdinalIgnoreCase.Equals(extension, ".sln"))
             {

src/Microsoft.DotNet.CodeFormatting/FormattingEngineImplementation.cs

@@ -34,6 +34,7 @@ internal sealed class FormattingEngineImplementation : IFormattingEngine
         private readonly IEnumerable<IGlobalSemanticFormattingRule> _globalSemanticRules;
         private readonly Stopwatch _watch = new Stopwatch();
         private bool _allowTables;
+        private bool _verbose;
 
         public ImmutableArray<string> CopyrightHeader
         {
@@ -64,6 +65,12 @@ public bool AllowTables
             get { return _allowTables; }
             set { _allowTables = value; }
         }
+
+        public bool Verbose
+        {
+            get { return _verbose; }
+            set { _verbose = value; }
+        }
 
         public bool ConvertUnicodeCharacters
         {
@@ -213,7 +220,10 @@ private void StartDocument()
         private void EndDocument(Document document)
         {
             _watch.Stop();
-            FormatLogger.WriteLine("    {0} {1} seconds", document.Name, _watch.Elapsed.TotalSeconds);
+            if (_verbose)
+            {
+                FormatLogger.WriteLine("    {0} {1} seconds", document.Name, _watch.Elapsed.TotalSeconds);
+            }
         }
 
         /// <summary>
@@ -224,7 +234,7 @@ private void EndDocument(Document document)
         /// </summary>
         private async Task<Solution> RunSyntaxPass(Solution originalSolution, IReadOnlyList<DocumentId> documentIds, CancellationToken cancellationToken)
         {
-            FormatLogger.WriteLine("Syntax Pass");
+            FormatLogger.WriteLine("\tSyntax Pass");
 
             var currentSolution = originalSolution;
             foreach (var documentId in documentIds)
@@ -264,7 +274,7 @@ private SyntaxNode RunSyntaxPass(SyntaxNode root, string languageName)
 
         private async Task<Solution> RunLocalSemanticPass(Solution solution, IReadOnlyList<DocumentId> documentIds, CancellationToken cancellationToken)
         {
-            FormatLogger.WriteLine("Local Semantic Pass");
+            FormatLogger.WriteLine("\tLocal Semantic Pass");
             foreach (var localSemanticRule in _localSemanticRules)
             {
                 solution = await RunLocalSemanticPass(solution, documentIds, localSemanticRule, cancellationToken);
@@ -275,7 +285,11 @@ private async Task<Solution> RunLocalSemanticPass(Solution solution, IReadOnlyLi
 
         private async Task<Solution> RunLocalSemanticPass(Solution originalSolution, IReadOnlyList<DocumentId> documentIds, ILocalSemanticFormattingRule localSemanticRule, CancellationToken cancellationToken)
         {
-            FormatLogger.WriteLine("  {0}", localSemanticRule.GetType().Name);
+            if (_verbose)
+            {
+                FormatLogger.WriteLine("  {0}", localSemanticRule.GetType().Name);
+            }
+
             var currentSolution = originalSolution;
             foreach (var documentId in documentIds)
             {
@@ -301,7 +315,7 @@ private async Task<Solution> RunLocalSemanticPass(Solution originalSolution, IRe
 
         private async Task<Solution> RunGlobalSemanticPass(Solution solution, IReadOnlyList<DocumentId> documentIds, CancellationToken cancellationToken)
         {
-            FormatLogger.WriteLine("Global Semantic Pass");
+            FormatLogger.WriteLine("\tGlobal Semantic Pass");
             foreach (var globalSemanticRule in _globalSemanticRules)
             {
                 solution = await RunGlobalSemanticPass(solution, documentIds, globalSemanticRule, cancellationToken);
@@ -312,7 +326,11 @@ private async Task<Solution> RunGlobalSemanticPass(Solution solution, IReadOnlyL
 
         private async Task<Solution> RunGlobalSemanticPass(Solution solution, IReadOnlyList<DocumentId> documentIds, IGlobalSemanticFormattingRule globalSemanticRule, CancellationToken cancellationToken)
         {
-            FormatLogger.WriteLine("  {0}", globalSemanticRule.GetType().Name);
+            if (_verbose)
+            {
+                FormatLogger.WriteLine("  {0}", globalSemanticRule.GetType().Name);
+            }
+
             foreach (var documentId in documentIds)
             {
                 var document = solution.GetDocument(documentId);

src/Microsoft.DotNet.CodeFormatting/IFormattingEngine.cs

@@ -17,6 +17,7 @@ public interface IFormattingEngine
         ImmutableArray<string> FileNames { get; set; }
         bool AllowTables { get; set; }
         bool ConvertUnicodeCharacters { get; set; }
+        bool Verbose { get; set; }
         Task FormatSolutionAsync(Solution solution, CancellationToken cancellationToken);
         Task FormatProjectAsync(Project porject, CancellationToken cancellationToken);
     }