[clang-tools-extra] [clang-tidy][NFC] Remove guidelines in documentation of misc-multiple-inheritance (PR #186657)
Zeyi Xu via cfe-commits
cfe-commits at lists.llvm.org
Sun Mar 15 04:43:37 PDT 2026
================
@@ -42,8 +42,3 @@ A class that inherits from a pure virtual is allowed:
virtual int foo() override { return 0; }
virtual int bar() override { return 0; }
};
-
-References
----------------
zeyi2 wrote:
I've removed the Fuchsia reference here, but I'd like to open a quick discussion to make sure this is exactly the direction we want to take.
IMHO having a "References" section for Fuchsia in a generic `misc` check isn't that ideal. However, while
the Fuchsia guidelines are still mentioned in the alias documentation (https://clang.llvm.org/extra/clang-tidy/checks/fuchsia/multiple-inheritance.html), that page has an automatic redirect to this rst file. From a user's perspective, they likely won't have enough time to actually read and click the guideline link before being redirected here.
I was wondering if we should follow the pattern used in [unused-return-value](https://clang.llvm.org/extra/clang-tidy/checks/bugprone/unused-return-value.html) and replace the "References" section with a single note like this:
> :doc:`fuchsia-multiple-inheritance <../fuchsia/multiple-inheritance>` is an alias of this check. See the features disallowed in Fuchsia at https://fuchsia.dev/fuchsia-src/development/languages/c-cpp/cxx?hl=en
I feel this might be a better way to preserve the necessary context for Fuchsia users without making it the primary reference for a generic check.
WDYT?
https://github.com/llvm/llvm-project/pull/186657
More information about the cfe-commits
mailing list