Add a property to return the number of lines in TextLines

bergmeister · Jun 16, 2017 · a1ec516 · a1ec516
1 parent d796058
commit a1ec516
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/Engine/EditableText.cs b/Engine/EditableText.cs
@@ -16,6 +16,11 @@ public class EditableText
     {
         private TextLines lines { get; set; }
 
+        /// <summary>
+        /// Return the number of lines in the text.
+        /// </summary>
+        public int LineCount => lines.Count;
+
         /// <summary>
         /// The text that is available for editing.
         /// </summary>

diff --git a/Engine/ScriptAnalyzer.cs b/Engine/ScriptAnalyzer.cs
@@ -1569,7 +1569,7 @@ public EditableText Fix(EditableText text, Range range, out Range updatedRange)
             }
 
             range = isRangeNull ? null : SnapToEdges(text, range);
-            var previousLineCount = text.Lines.Length;
+            var previousLineCount = text.LineCount;
             var previousUnusedCorrections = 0;
             do
             {
@@ -1600,7 +1600,7 @@ 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.Lines.Length;
+                var lineCount = text.LineCount;
                 if (!isRangeNull && lineCount != previousLineCount)
                 {
                     range = new Range(