On May 29, 2014, at 20:24 , Richard Smith <[email protected]> wrote:
> On Tue, May 27, 2014 at 8:22 PM, Jordan Rose <[email protected]> wrote: > Wow, this is showing me how many of the analyzer checks aren't handling > namespaces within std. > > I think they might all be correct: the last two walk up to the outermost > namespace before checking whether it's 'std'. The first of the three is > looking for exactly 'std::string'. Ah, for some reason I was ignoring the rest of the function. The LLVMConventionsChecker is way out of date, but checking for "std::string" exactly isn't going to work with libc++. We should probably just remove that checker until someone feels like reviving it and starting to use it regularly. Jordan
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
