[clang-tools-extra] [clang-tidy] Add check hicpp-ignored-remove-result (PR #73119)

Piotr Zegar via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 23 02:09:12 PST 2023


=?utf-8?q?Björn?= Svensson <bjorn.a.svensson at est.tech>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/73119 at github.com>


https://github.com/PiotrZSL commented:

Looks fine, as we got already such checks that are and are not aliases.
But consider doing this as an actually separate check instead of using an alias functionality.
Simply change original check, add other constructor, split storeOptions method, so in this check list of functions would not be configurable by end user.

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


More information about the cfe-commits mailing list