Fix for multiple projects

jaredpar · jaredpar · commit 32fec62c2f72 · 2015-02-22T08:03:06.000-08:00
The original fix only handled the parse change for single projects, this
fixes it to work for all projects.
diff --git a/src/Microsoft.DotNet.CodeFormatting/FormattingEngineImplementation.cs b/src/Microsoft.DotNet.CodeFormatting/FormattingEngineImplementation.cs
@@ -128,16 +128,15 @@ private Solution AddTablePreprocessorSymbol(Solution solution)
         /// </summary>
         private Solution RemoveTablePreprocessorSymbol(Solution newSolution, Solution oldSolution)
         {
-            var solution = newSolution;
-            var projectIds = solution.ProjectIds;
+            var projectIds = newSolution.ProjectIds;
             foreach (var projectId in projectIds)
             {
                 var oldProject = oldSolution.GetProject(projectId);
                 var newProject = newSolution.GetProject(projectId);
-                solution = newProject.WithParseOptions(oldProject.ParseOptions).Solution;
+                newSolution = newProject.WithParseOptions(oldProject.ParseOptions).Solution;
             }
 
-            return solution;
+            return newSolution;
         }
 
         internal async Task<Solution> FormatCoreAsync(Solution originalSolution, IReadOnlyList<DocumentId> documentIds, CancellationToken cancellationToken)

Original file line number	Diff line number	Diff line change
`@@ -128,16 +128,15 @@ private Solution AddTablePreprocessorSymbol(Solution solution)`
`128`	`128`	`/// </summary>`
`129`	`129`	`private Solution RemoveTablePreprocessorSymbol(Solution newSolution, Solution oldSolution)`
`130`	`130`	`{`
`131`		`- var solution = newSolution;`
`132`		`- var projectIds = solution.ProjectIds;`
	`131`	`+ var projectIds = newSolution.ProjectIds;`
`133`	`132`	`foreach (var projectId in projectIds)`
`134`	`133`	`{`
`135`	`134`	`var oldProject = oldSolution.GetProject(projectId);`
`136`	`135`	`var newProject = newSolution.GetProject(projectId);`
`137`		`- solution = newProject.WithParseOptions(oldProject.ParseOptions).Solution;`
	`136`	`+ newSolution = newProject.WithParseOptions(oldProject.ParseOptions).Solution;`
`138`	`137`	`}`
`139`	`138`
`140`		`- return solution;`
	`139`	`+ return newSolution;`
`141`	`140`	`}`
`142`	`141`
`143`	`142`	`internal async Task<Solution> FormatCoreAsync(Solution originalSolution, IReadOnlyList<DocumentId> documentIds, CancellationToken cancellationToken)`