More curly braces and less HashSet.Contains invocations.

ido-namely · Feb 6, 2018 · 046bc0c · 046bc0c
1 parent 2ff5f26
commit 046bc0c
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/src/MiniCover/Reports/HtmlReport.cs b/src/MiniCover/Reports/HtmlReport.cs
@@ -56,16 +56,20 @@ protected override void WriteDetailedReport(InstrumentationResult result, IDicti
                     htmlWriter.WriteLine("<body style=\"font-family: sans-serif;\">");
 
                     var uncoveredLineNumbers = new HashSet<int>();
-                    foreach(var i in kvFile.Value.Instructions)
-                        if (!hits.Contains(i.Id))
-                            for (var lineIndex = i.StartLine; lineIndex <= i.EndLine; lineIndex++)
-                                uncoveredLineNumbers.Add(lineIndex);
-
                     var coveredLineNumbers = new HashSet<int>();
-                    foreach (var i in kvFile.Value.Instructions)
+                    foreach(var i in kvFile.Value.Instructions)
+                    {
                         if (hits.Contains(i.Id))
+                        {
                             for (var lineIndex = i.StartLine; lineIndex <= i.EndLine; lineIndex++)
                                 coveredLineNumbers.Add(lineIndex);
+                        }
+                        else
+                        {
+                            for (var lineIndex = i.StartLine; lineIndex <= i.EndLine; lineIndex++)
+                                uncoveredLineNumbers.Add(lineIndex);
+                        }
+                    }
 
                     var l = 0;
                     foreach (var line in lines)