Missed a minor optimization as I wasn't using a parameter for the binary search method. The files I've opened don't actually hit the code path (they always hit the optimization for checking the previous/next/current line)

ToddGrun · ToddGrun · commit c027fcf8e60f · 2020-05-07T20:44:15.000-07:00
\n\nCommit migrated from dotnet/razor@75f34ee
diff --git a/src/Razor/Microsoft.AspNetCore.Razor.Language/src/Legacy/LineTrackingStringBuffer.cs b/src/Razor/Microsoft.AspNetCore.Razor.Language/src/Legacy/LineTrackingStringBuffer.cs
@@ -138,7 +138,7 @@ private TextLine ScanLines(int absoluteIndex, int startLineIndex, int endLineInd
         {
             // binary search for the line containing absoluteIndex
             var lowIndex = startLineIndex;
-            var highIndex = _lines.Count;
+            var highIndex = endLineIndex;
 
             while (lowIndex != highIndex)
             {

Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,7 @@ private TextLine ScanLines(int absoluteIndex, int startLineIndex, int endLineInd`
`138`	`138`	`{`
`139`	`139`	`// binary search for the line containing absoluteIndex`
`140`	`140`	`var lowIndex = startLineIndex;`
`141`		`- var highIndex = _lines.Count;`
	`141`	`+ var highIndex = endLineIndex;`
`142`	`142`
`143`	`143`	`while (lowIndex != highIndex)`
`144`	`144`	`{`