On Apr 10, 2012, at 8:49 PM, Nico Weber wrote: > 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.
Besides, wouldn't it be better to mark it as "pure"? -Chris _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
