Basic structure working

The new code is actually so fast that it is hard to keep up with the
output.  It can run very quickly on the CodeAnalysis project now except
there is a bug in private field rename.  Need to cleanup the output to
be useable again.  Then onto debugging that issue.

Author: jaredpar

src/Microsoft.DotNet.CodeFormatting/FormattingEngineImplementation.cs

@@ -52,6 +52,9 @@ public Task FormatProjectAsync(Project project, CancellationToken cancellationTo
 
         private async Task FormatAsync(Workspace workspace, IReadOnlyList<DocumentId> documentIds, CancellationToken cancellationToken)
         {
+            var watch = new Stopwatch();
+            watch.Start();
+
             var originalSolution = workspace.CurrentSolution;
             var solution = originalSolution;
             solution = await RunSyntaxPass(solution, documentIds, cancellationToken);
@@ -73,6 +76,9 @@ private async Task FormatAsync(Workspace workspace, IReadOnlyList<DocumentId> do
                     }
                 }
             }
+
+            watch.Stop();
+            Console.WriteLine("Total time {0}", watch.Elapsed);
         }
 
         private async Task<bool> ShouldBeProcessedAsync(Document document)
@@ -101,7 +107,7 @@ private void StartDocument(Document document, int depth = 1)
         {
             for (int i = 0; i < depth; i++)
             {
-                Console.Write("\t");
+                Console.Write("  ");
             }
 
             Console.Write("Processing {0}", document.Name);
@@ -175,7 +181,7 @@ private async Task<Solution> RunLocalSemanticPass(Solution solution, IReadOnlyLi
 
         private async Task<Solution> RunLocalSemanticPass(Solution originalSolution, IReadOnlyList<DocumentId> documentIds, ILocalSemanticFormattingRule localSemanticRule, CancellationToken cancellationToken)
         {
-            Console.WriteLine("\t{0}", localSemanticRule.GetType().Name);
+            Console.WriteLine("  {0}", localSemanticRule.GetType().Name);
             var currentSolution = originalSolution;
             foreach (var documentId in documentIds)
             {
@@ -212,7 +218,7 @@ private async Task<Solution> RunGlobalSemanticPass(Solution solution, IReadOnlyL
 
         private async Task<Solution> RunGlobalSemanticPass(Solution solution, IReadOnlyList<DocumentId> documentIds, IGlobalSemanticFormattingRule globalSemanticRule, CancellationToken cancellationToken)
         {
-            Console.WriteLine("\t{0}", globalSemanticRule.GetType().Name);
+            Console.WriteLine("  {0}", globalSemanticRule.GetType().Name);
             foreach (var documentId in documentIds)
             {
                 var document = solution.GetDocument(documentId);

src/Microsoft.DotNet.CodeFormatting/IFormattingRule.cs

@@ -18,7 +18,7 @@ internal interface IFormattingRule
     /// <summary>
     /// Rules which need no semantic information and operate on parse trees only.  
     /// </summary>
-    internal interface ISyntaxFormattingRule
+    internal interface ISyntaxFormattingRule : IFormattingRule
     {
         SyntaxNode Process(SyntaxNode syntaxRoot);
     }
@@ -28,15 +28,15 @@ internal interface ISyntaxFormattingRule
     /// used for rules that need to see a <see cref="Document"/> and <see cref="SyntaxNode"/> which
     /// are in sync with each other,
     /// </summary>
-    internal interface ILocalSemanticFormattingRule
+    internal interface ILocalSemanticFormattingRule : IFormattingRule
     {
         Task<SyntaxNode> ProcessAsync(Document document, SyntaxNode syntaxRoot, CancellationToken cancellationToken);
     }
 
     /// <summary>
     /// Rules which can affect more than the local document
     /// </summary>
-    internal interface IGlobalSemanticFormattingRule
+    internal interface IGlobalSemanticFormattingRule : IFormattingRule
     {
         Task<Solution> ProcessAsync(Document document, SyntaxNode syntaxRoot, CancellationToken cancellationToken);
     }

src/Microsoft.DotNet.CodeFormatting/Rules/HasNoIllegalHeadersFormattingRule.cs

@@ -15,7 +15,7 @@
 
 namespace Microsoft.DotNet.CodeFormatting.Rules
 {
-    [RuleOrder(RuleOrder.HasNoIllegalHeadersFormattingRule)]
+    // [RuleOrder(RuleOrder.HasNoIllegalHeadersFormattingRule)]
     internal sealed class HasNoIllegalHeadersFormattingRule 
     {
         // We are going to replace this header with the actual filename of the document being processed