[Frontend] Use StringRef::drop_while (NFC)

kazutakahirata · kazutakahirata · commit 5c37e711df6b · 2023-12-28T10:06:20.000-08:00
diff --git a/clang/lib/Frontend/LayoutOverrideSource.cpp b/clang/lib/Frontend/LayoutOverrideSource.cpp
@@ -147,8 +147,7 @@ LayoutOverrideSource::LayoutOverrideSource(StringRef Filename) {
 
         // Skip over this offset, the following comma, and any spaces.
         LineStr = LineStr.substr(1);
-        while (!LineStr.empty() && isWhitespace(LineStr[0]))
-          LineStr = LineStr.substr(1);
+        LineStr = LineStr.drop_while(isWhitespace);
       }
     }
 
@@ -163,8 +162,7 @@ LayoutOverrideSource::LayoutOverrideSource(StringRef Filename) {
 
         // Skip over this offset, the following comma, and any spaces.
         LineStr = LineStr.substr(1);
-        while (!LineStr.empty() && isWhitespace(LineStr[0]))
-          LineStr = LineStr.substr(1);
+        LineStr = LineStr.drop_while(isWhitespace);
       }
       continue;
     }
@@ -180,8 +178,7 @@ LayoutOverrideSource::LayoutOverrideSource(StringRef Filename) {
 
         // Skip over this offset, the following comma, and any spaces.
         LineStr = LineStr.substr(1);
-        while (!LineStr.empty() && isWhitespace(LineStr[0]))
-          LineStr = LineStr.substr(1);
+        LineStr = LineStr.drop_while(isWhitespace);
       }
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -147,8 +147,7 @@ LayoutOverrideSource::LayoutOverrideSource(StringRef Filename) {`
`147`	`147`
`148`	`148`	`// Skip over this offset, the following comma, and any spaces.`
`149`	`149`	`LineStr = LineStr.substr(1);`
`150`		`- while (!LineStr.empty() && isWhitespace(LineStr[0]))`
`151`		`- LineStr = LineStr.substr(1);`
	`150`	`+ LineStr = LineStr.drop_while(isWhitespace);`
`152`	`151`	`}`
`153`	`152`	`}`
`154`	`153`
`@@ -163,8 +162,7 @@ LayoutOverrideSource::LayoutOverrideSource(StringRef Filename) {`
`163`	`162`
`164`	`163`	`// Skip over this offset, the following comma, and any spaces.`
`165`	`164`	`LineStr = LineStr.substr(1);`
`166`		`- while (!LineStr.empty() && isWhitespace(LineStr[0]))`
`167`		`- LineStr = LineStr.substr(1);`
	`165`	`+ LineStr = LineStr.drop_while(isWhitespace);`
`168`	`166`	`}`
`169`	`167`	`continue;`
`170`	`168`	`}`
`@@ -180,8 +178,7 @@ LayoutOverrideSource::LayoutOverrideSource(StringRef Filename) {`
`180`	`178`
`181`	`179`	`// Skip over this offset, the following comma, and any spaces.`
`182`	`180`	`LineStr = LineStr.substr(1);`
`183`		`- while (!LineStr.empty() && isWhitespace(LineStr[0]))`
`184`		`- LineStr = LineStr.substr(1);`
	`181`	`+ LineStr = LineStr.drop_while(isWhitespace);`
`185`	`182`	`}`
`186`	`183`	`}`
`187`	`184`	`}`