[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