Szelethus added inline comments.
================ Comment at: docs/analyzer/checkers.rst:225-226 +``std::string``s, by recognizing member functions that may re/deallocate the buffer +before use. In the future, it would be great to add support for other STL and +non-STL containers, and most notably, ``std::string_view``s. + ---------------- dkrupp wrote: > Szelethus wrote: > > Hmm. While this page is a documentation, I would still expect regular users > > to browse through it -- are we sure that we want to add future plans for a > > non-alpha checker? I'm not against it, just a question. > I think it is a good idea. A non-alpha checker can also be further developed, > by anyone else. It is good that we don't forget about further features. This > note also highlights the limitations of the checker. How about this: "Future plans include to add support for blahblah". The current statement should rather be a TODO in the code. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60281/new/ https://reviews.llvm.org/D60281 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
