Update method to normalize range

Author: Kapil Borle

Engine/Range.cs

@@ -43,26 +43,26 @@ public Range Shift(int lineDelta, int columnDelta)
             return new Range(newStart, newEnd);
         }
 
-        public static Range Normalize(Range refRange, Range range)
+        public static Range Normalize(Position refPosition, Range range)
         {
-            if (refRange == null)
+            if (refPosition == null)
             {
-                throw new ArgumentNullException(nameof(refRange));
+                throw new ArgumentNullException(nameof(refPosition));
             }
 
             if (range == null)
             {
                 throw new ArgumentNullException(nameof(range));
             }
 
-            if (refRange.Start > range.Start)
+            if (refPosition > range.Start)
             {
                 throw new ArgumentException("reference range should start before range");
             }
 
             return range.Shift(
-                1 - refRange.Start.Line,
-                range.Start.Line == refRange.Start.Line ? 1 - refRange.Start.Column : 0);
+                1 - refPosition.Line,
+                range.Start.Line == refPosition.Line ? 1 - refPosition.Column : 0);
         }
     }
 }

rules/UseSupportsShouldProcess.cs

@@ -193,9 +193,10 @@ private List<CorrectionExtent> GetCorrections(
             });
 
             var editableText = new EditableText(funcDefnAst.Extent.Text);
+            var funcDefAstStartPos = funcDefnAst.Extent.ToRange().Start;
             foreach (var correctionExtent in correctionExtents)
             {
-                var shiftedCorrectionExtent = Normalize(funcDefnAst.Extent, correctionExtent);
+                var shiftedCorrectionExtent = Normalize(funcDefAstStartPos, correctionExtent);
                 editableText = editableText.ApplyEdit1(shiftedCorrectionExtent);
             }
 
@@ -281,13 +282,10 @@ private static CorrectionExtent GetCorrectionToRemoveFuncParamDecl(
 
         // doesn't seem right. The arguments should be of same type.
         private CorrectionExtent Normalize(
-            IScriptExtent referenceExtent,
+            Position refPosition,
             CorrectionExtent correctionExtent)
         {
-            var shiftedRange = Range.Normalize(referenceExtent.ToRange(), correctionExtent);
-
-            // TODO Add a method to TextEdit class that takes in range and text
-            // TODO Add a method to CorrectionExtent that takes in range and all other stuff
+            var shiftedRange = Range.Normalize(refPosition, (Range)correctionExtent);
             return new CorrectionExtent(
                  shiftedRange.Start.Line,
                  shiftedRange.End.Line,