Hi,

the attached patch marks vector:: and string::empty() as
warn_unused_result. It looks like it's fairly often confused with
clear() in practice:

https://breakpad.appspot.com/377003/
http://codereview.chromium.org/10034006/
http://www.viva64.com/en/a/0079/ (search for "empty")

See also http://llvm.org/bugs/show_bug.cgi?id=10011 for a more general approach.

Nico

Attachment: libcxx-unused.patch
Description: Binary data

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to