[Parse] Remove unnecessary conversion to SourceRange (swiftlang#28926)

dduan · web-flow · commit 0c6c5aa1a2c1 · 2020-01-02T11:05:30.000-08:00
`SourceLoc`s are implicitly converted to `SourceRange`. Yay.

NFC, Gardening
diff --git a/lib/Parse/Parser.cpp b/lib/Parse/Parser.cpp
@@ -1054,7 +1054,7 @@ Parser::parseList(tok RightK, SourceLoc LeftLoc, SourceLoc &RightLoc,
   while (true) {
     while (Tok.is(tok::comma)) {
       diagnose(Tok, diag::unexpected_separator, ",")
-        .fixItRemove(SourceRange(Tok.getLoc()));
+        .fixItRemove(Tok.getLoc());
       consumeToken();
     }
     SourceLoc StartLoc = Tok.getLoc();
@@ -1084,7 +1084,7 @@ Parser::parseList(tok RightK, SourceLoc LeftLoc, SourceLoc &RightLoc,
         continue;
       if (!AllowSepAfterLast) {
         diagnose(Tok, diag::unexpected_separator, ",")
-          .fixItRemove(SourceRange(PreviousLoc));
+          .fixItRemove(PreviousLoc);
       }
       break;
     }

Original file line number	Diff line number	Diff line change
`@@ -1054,7 +1054,7 @@ Parser::parseList(tok RightK, SourceLoc LeftLoc, SourceLoc &RightLoc,`
`1054`	`1054`	`while (true) {`
`1055`	`1055`	`while (Tok.is(tok::comma)) {`
`1056`	`1056`	`diagnose(Tok, diag::unexpected_separator, ",")`
`1057`		`- .fixItRemove(SourceRange(Tok.getLoc()));`
	`1057`	`+ .fixItRemove(Tok.getLoc());`
`1058`	`1058`	`consumeToken();`
`1059`	`1059`	`}`
`1060`	`1060`	`SourceLoc StartLoc = Tok.getLoc();`
`@@ -1084,7 +1084,7 @@ Parser::parseList(tok RightK, SourceLoc LeftLoc, SourceLoc &RightLoc,`
`1084`	`1084`	`continue;`
`1085`	`1085`	`if (!AllowSepAfterLast) {`
`1086`	`1086`	`diagnose(Tok, diag::unexpected_separator, ",")`
`1087`		`- .fixItRemove(SourceRange(PreviousLoc));`
	`1087`	`+ .fixItRemove(PreviousLoc);`
`1088`	`1088`	`}`
`1089`	`1089`	`break;`
`1090`	`1090`	`}`