Szelethus added inline comments.

Comment at: docs/analyzer/checkers.rst:225-226
+``std::string``s, by recognizing member functions that may re/deallocate the 
+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.

  rC Clang


cfe-commits mailing list

Reply via email to