reduce diff

Author: edgarfgp

src/Compiler/Checking/CheckRecordSyntaxHelpers.fs

@@ -108,8 +108,7 @@ let TransformAstForNestedUpdates
 
         match withExpr with
         | SynExpr.Ident origId, (blockSep: BlockSeparator) ->
-            let sepRange = blockSep.Range
-            let lid, rng = upToId sepRange id (origId :: ids)
+            let lid, rng = upToId blockSep.Range id (origId :: ids)
             // We need a neutral, “offside” separator for the AST that does not claim there was a concrete token like a semicolon or a comma
             Some(
                 SynExpr.LongIdent(false, LongIdentWithDots(lid, rng), None, totalRange origId id),

src/Compiler/Service/ServiceParseTreeWalk.fs

@@ -448,11 +448,9 @@ module SyntaxTraversal =
                         | Some(expr, blockSep) ->
                             yield dive expr expr.Range traverseSynExpr
 
-                            let withRange = blockSep.Range
-
                             yield
-                                dive () withRange (fun () ->
-                                    if posGeq pos withRange.End then
+                                dive () blockSep.Range (fun () ->
+                                    if posGeq pos blockSep.Range.End then
                                         // special case: caret is after WITH
                                         // { x with $ }
                                         visitor.VisitRecordField(path, Some expr, None)
@@ -518,11 +516,9 @@ module SyntaxTraversal =
                         | Some(expr, blockSep) ->
                             yield dive expr expr.Range traverseSynExpr
 
-                            let withRange = blockSep.Range
-
                             yield
-                                dive () withRange (fun () ->
-                                    if posGeq pos withRange.End then
+                                dive () blockSep.Range (fun () ->
+                                    if posGeq pos blockSep.Range.End then
                                         // special case: caret is after WITH
                                         // { x with $ }
                                         visitor.VisitRecordField(path, Some expr, None)
@@ -564,16 +560,14 @@ module SyntaxTraversal =
 
                             match sepOpt with
                             | Some sep ->
-                                let scPosOpt = sep.Position
-                                let sepRange = sep.Range
 
                                 yield
-                                    dive () sepRange (fun () ->
+                                    dive () sep.Range (fun () ->
                                         // special case: caret is between field bindings
                                         // field1 = 5
                                         // $
                                         // field2 = 5
-                                        diveIntoSeparator offsideColumn scPosOpt copyOpt)
+                                        diveIntoSeparator offsideColumn sep.Position copyOpt)
                             | _ -> ()
 
                     ]