jvoung 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. 
> Pinging @tstellar for clarification.

Simplified the mock, since we're only matching on the top-level macro names.

https://github.com/llvm/llvm-project/pull/115051
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to