[clang-tools-extra] [clang-tidy] Filter out googletest TUs in bugprone-unchecked-optional-access (PR #115051)

Carlos Galvez via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 6 11:26:30 PST 2024


carlosgalvezp wrote:

> but these are not internal implementation details - these are key elements of the public API 

In the unit test, you have copied internal code from here:
https://github.com/google/googletest/blob/d144031940543e15423a25ae5a8a74141044862f/googletest/include/gtest/internal/gtest-internal.h#L1477

This is not OK to do, since it may change at any time and make the test no longer useful. The test may also work for some versions of googletest but not others.

The code that has been copied is also Copyrighted and has certain License obligations, which we may not be allowed to bring into the LLVM repository.

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


More information about the cfe-commits mailing list