On Jul 26, 2011, at 10:58 AM, Fariborz Jahanian wrote: > Author: fjahanian > Date: Tue Jul 26 12:58:54 2011 > New Revision: 136103 > > URL: http://llvm.org/viewvc/llvm-project?rev=136103&view=rev > Log: > Provide fixit for static use of objective-c type > in few more places and in each instance, fix up > the type to the expected type. // rdar://9603056
One comment about the test below. > Modified: cfe/trunk/test/FixIt/fixit-static-object-decl.m > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/test/FixIt/fixit-static-object-decl.m?rev=136103&r1=136102&r2=136103&view=diff > ============================================================================== > --- cfe/trunk/test/FixIt/fixit-static-object-decl.m (original) > +++ cfe/trunk/test/FixIt/fixit-static-object-decl.m Tue Jul 26 12:58:54 2011 > @@ -9,10 +9,21 @@ > // RUN: %clang_cc1 -fsyntax-only -Werror -x objective-c++ %t > // rdar://9603056 > > +@interface S @end > + > @interface NSArray > +{ > +@public > + S iS; > +} > + (id) arrayWithObjects; > @end > > +NSArray func() { > + NSArray P; > + return P; > +} > + > int main() { > NSArray pluginNames = [NSArray arrayWithObjects]; > } Some of the other tests in test/FixIt actually use the -fixit mode to write out a modified source file, and then verify that the source file parses without any errors. It's a great way to ensure that the location information in the fix is accurate. - Doug _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
