I see no tests in your patch, please add a test. Also, you seem to allow %k/%K in any context, not just ones where ObjCRuntime::getKind() == ObjFW. Please fix this.
On Sat, Nov 23, 2013 at 10:03 AM, Jonathan Schleifer <[email protected]> wrote: > Please see the updated attached patch. It also adds the %K format > specifier, which is for an array of unicode characters terminated with 0 > instead of a single one. > > Again, please CC me as I'm not on the list. > > -- > Jonathan > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
