This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [Patch, ObjC, test-suite, committed] "next-ify" recent ObjC test-suite additions.



On 4 Nov 2010, at 18:45, Nicola Pero wrote:



Shall we remove the #ifdef __NEXT_RUNTIME__ and always add a +initialize in all tests
for both runtimes ? ie, in all tests we can simply add


+ (id) initialize;

+ (id) initialize { return self; }

as the first method in all classes. Since the +initialize is not part of the test, I
find it less distracting to remove the #ifdefs and give it as little visual prominence
as possible, since it's almost never relevant to what is being tested.


Ie, I'm happy to agree that we'll add +initialize in all tests for the GNU runtime as well,
and we can drop the #ifdefs. :-)

agreed, the fewer ifdefs, the better.


I've got a similar job to do on ObjC++ ... hopefully this evening.

cheers
Iain


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]