JonasToth added inline comments.
================ Comment at: clang-tidy/bugprone/UnusedReturnValueCheck.cpp:47 + "^::std::launder$|" + "^::std::unique_ptr<.*>::release$|" + "^::std::.*::allocate$|" ---------------- khuttun wrote: > JonasToth wrote: > > Is the following type a problem for you check? > > > > `std::unique_ptr<std::vector<int>>` should not be matchable with regex but > > I don't know if that would have an impact on the functionality. > `std::unique_ptr<std::vector<int>>` is also matched. I added a test case for > it. Alright. I think i had a error in my thinking. Because there are followup letters in the regex there are no problems. I thought it would end on the first `>`, but thats no the case! Thank you for clarification :) https://reviews.llvm.org/D41655 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
