Thanks! Landed r160886. - Hans
On Fri, Jul 27, 2012 at 7:15 PM, Ted Kremenek <[email protected]> wrote: > Looks great to me. Thanks for doing this! > > On Jul 27, 2012, at 9:38 AM, Matt Beaumont-Gay <[email protected]> wrote: > > LGTM, but maybe Doug or Ted should comment on whether this is actually the > behavior we want. > > > On Fri, Jul 27, 2012 at 6:17 AM, Hans Wennborg <[email protected]> wrote: >> >> Hi all, >> >> Clang's -Wformat fix-its currently suggest using "%zu" for values of >> type size_t (in C99 and C++11). >> >> However, for a type such as std::vector<T>::size_type, it doesn't >> notice that this is typedeffed to size_t, and therefore it doesn't >> suggest "%zu", but rather "%u" or "%lu". >> >> The attached patch makes the printf/scanf fix-up mechanism walk the >> typedef chain, so that it notices if the type is size_t, even if that >> isn't "at the top". >> >> Please take a look. _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
