ArcsinX marked 2 inline comments as done.
ArcsinX added inline comments.
================
Comment at: clang-tools-extra/clangd/unittests/TypeHierarchyTests.cpp:533
+
+ Annotations HeaderInPreambleAnnotations(R"cpp(
+struct Parent {
----------------
kadircet wrote:
> this doesn't need to be an `Annotation` and sorry for missing it in the first
> pass but since this only has a single header, you can actually do something
> like:
>
> ```
> TestTU TU = TestTU::withCode(SourceAnnotations.code());
> TU.HeaderCode = "struct Parent { int a; }";
> ```
>
> and also drop the include directive in `SourceAnnotations` as `TU.HeaderCode`
> is implicitly included.
Seems now I need it to check selection range.
================
Comment at: clang-tools-extra/clangd/unittests/TypeHierarchyTests.cpp:548
+ EXPECT_THAT(*Result, AllOf(WithName("Child"),
+ Parents(AllOf(WithName("Parent"), Parents()))));
+}
----------------
kadircet wrote:
> could you also make sure selection range is correct at least for `Parent` (as
> main file ranges are tested elsewhere already), so that we don't regress it
> in the future.
Added Parent selection range check.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D84144/new/
https://reviews.llvm.org/D84144
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
