[clang-tools-extra] Add support for renaming objc methods, even those with multiple selector pieces (PR #76466)

kadir çetinkaya via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 5 02:38:56 PST 2024


================
@@ -508,7 +508,8 @@ static bool mayBeValidIdentifier(llvm::StringRef Ident) {
       !isAsciiIdentifierStart(Ident.front(), AllowDollar))
     return false;
   for (char C : Ident) {
-    if (llvm::isASCII(C) && !isAsciiIdentifierContinue(C, AllowDollar))
+    if (llvm::isASCII(C) && !isAsciiIdentifierContinue(C, AllowDollar) &&
----------------
kadircet wrote:

nit:
```cpp
if (AllowColon && C == ':')
  continue;
if (llvm::isASCII ...)
```

https://github.com/llvm/llvm-project/pull/76466


More information about the cfe-commits mailing list