Remove todo comments from ScriptAnalyzer.Fix

Kapil Borle · Kapil Borle · commit 22015c9f80b8 · 2017-06-16T11:03:02.000-07:00
diff --git a/Engine/ScriptAnalyzer.cs b/Engine/ScriptAnalyzer.cs
@@ -1557,7 +1557,6 @@ public EditableText Fix(EditableText text, Range range, out Range updatedRange)
                 throw new ArgumentNullException(nameof(text));
             }
 
-            // todo validate range
             var isRangeNull = range == null;
             if (!isRangeNull && !text.IsValidRange(range))
             {
@@ -1598,8 +1597,6 @@ public EditableText Fix(EditableText text, Range range, out Range updatedRange)
                 }
 
                 previousUnusedCorrections = unusedCorrections;
-
-                // todo add a TextLines.NumLines property because accessing TextLines.Lines is expensive
                 var lineCount = text.LineCount;
                 if (!isRangeNull && lineCount != previousLineCount)
                 {

Original file line number	Diff line number	Diff line change
`@@ -1557,7 +1557,6 @@ public EditableText Fix(EditableText text, Range range, out Range updatedRange)`
`1557`	`1557`	`throw new ArgumentNullException(nameof(text));`
`1558`	`1558`	`}`
`1559`	`1559`
`1560`		`- // todo validate range`
`1561`	`1560`	`var isRangeNull = range == null;`
`1562`	`1561`	`if (!isRangeNull && !text.IsValidRange(range))`
`1563`	`1562`	`{`
`@@ -1598,8 +1597,6 @@ public EditableText Fix(EditableText text, Range range, out Range updatedRange)`
`1598`	`1597`	`}`
`1599`	`1598`
`1600`	`1599`	`previousUnusedCorrections = unusedCorrections;`
`1601`		`-`
`1602`		`- // todo add a TextLines.NumLines property because accessing TextLines.Lines is expensive`
`1603`	`1600`	`var lineCount = text.LineCount;`
`1604`	`1601`	`if (!isRangeNull && lineCount != previousLineCount)`
`1605`	`1602`	`{`