ymandel marked an inline comment as done. ymandel added inline comments.
================ Comment at: clang-tools-extra/unittests/clang-tidy/TransformerClangTidyCheckTest.cpp:106 +// argument, while the change spans only the argument AND there are two such +// matches. Here, we expect a conflict between the two matches and the second +// to be ignored. ---------------- gribozavr wrote: > Sorry, I don't quite understand the comment -- the test has two replacements > (both arguments to the macro are replaced), but the comment says that we only > expect one. Right, that comment needs to be updated. It's leftover from while the fix wasn't done yet... Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66676/new/ https://reviews.llvm.org/D66676 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
