Quolyk added a comment. For now, I just added tests. I have several questions, as I'm beginner (`ContainerSizeEmptyCheck.cpp`).
1. Do I have to extend `ValidContainer`, so it recognises `::std::string` with `length()` method as valid container too or we can assume `::std::string` as valid container by default? 2. If `::std::string` is valid container, I just add one more expression to match. Is it correct way? Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56644/new/ https://reviews.llvm.org/D56644 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
