[clang-tools-extra] [clangd] Update XRefs to support overriden ObjC methods (PR #127109)

via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 13 10:57:44 PST 2025


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff 859c871184bdfdebb47b5c7ec5e59348e0534e0b d335931b8923005d2ffc580c5195b7fa997f3649 --extensions cpp -- clang-tools-extra/clangd/XRefs.cpp clang-tools-extra/clangd/unittests/XRefsTests.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/clang-tools-extra/clangd/unittests/XRefsTests.cpp b/clang-tools-extra/clangd/unittests/XRefsTests.cpp
index a310a51cf4..0f8d096e28 100644
--- a/clang-tools-extra/clangd/unittests/XRefsTests.cpp
+++ b/clang-tools-extra/clangd/unittests/XRefsTests.cpp
@@ -426,9 +426,10 @@ TEST(LocateSymbol, FindOverridesObjC) {
   TestTU TU = TestTU::withCode(Code.code());
   TU.ExtraArgs.push_back("-xobjective-c++");
   auto AST = TU.build();
-  EXPECT_THAT(locateSymbolAt(AST, Code.point(), TU.index().get()),
-              UnorderedElementsAre(sym("foo", Code.range("1"), std::nullopt),
-                                   sym("foo", Code.range("2"), Code.range("2"))));
+  EXPECT_THAT(
+      locateSymbolAt(AST, Code.point(), TU.index().get()),
+      UnorderedElementsAre(sym("foo", Code.range("1"), std::nullopt),
+                           sym("foo", Code.range("2"), Code.range("2"))));
 }
 
 TEST(LocateSymbol, WithIndexPreferredLocation) {
@@ -1878,15 +1879,15 @@ TEST(FindImplementations, InheritanceObjC) {
   TU.ExtraArgs.push_back("-xobjective-c++");
   auto AST = TU.build();
   auto Index = TU.index();
-  EXPECT_THAT(
-    findImplementations(AST, Code.point("base"), Index.get()),
-    UnorderedElementsAre(sym("Child", Code.range("ChildDecl"), Code.range("ChildDef"))));
-  EXPECT_THAT(
-    findImplementations(AST, Code.point("foo"), Index.get()),
-    UnorderedElementsAre(sym("foo", Code.range("fooDecl"), Code.range("fooDef"))));
-  EXPECT_THAT(
-    findImplementations(AST, Code.point("protocol"), Index.get()),
-    UnorderedElementsAre(sym("protocol", Code.range("protocolDef"), Code.range("protocolDef"))));
+  EXPECT_THAT(findImplementations(AST, Code.point("base"), Index.get()),
+              UnorderedElementsAre(sym("Child", Code.range("ChildDecl"),
+                                       Code.range("ChildDef"))));
+  EXPECT_THAT(findImplementations(AST, Code.point("foo"), Index.get()),
+              UnorderedElementsAre(
+                  sym("foo", Code.range("fooDecl"), Code.range("fooDef"))));
+  EXPECT_THAT(findImplementations(AST, Code.point("protocol"), Index.get()),
+              UnorderedElementsAre(sym("protocol", Code.range("protocolDef"),
+                                       Code.range("protocolDef"))));
 }
 
 TEST(FindImplementations, CaptureDefinition) {

``````````

</details>


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


More information about the cfe-commits mailing list