Wrong argument type for NSAssert1

Question

This isn't giving me an error but several warnings I'd like to get rid of.

OSErr err = AudioUnitInitialize(toneUnit);
NSAssert1(err == noErr, @"Error initializing unit: %ld", err);

The warning I'm getting is: Format specifies type 'long' but the argument has type 'OSErr' (aka 'short'). Obviously I can change this by using long instead of OSErr but I was wondering why I'm getting the error. This is an upcast so I thought it shouldn't cause any problems.

Can anyone explain?

just a heads up - dont use NSAssert1, NSAssert2 etc. Just use NSAssert — hooleyhoop, Aug 03 '12 at 11:18
You don't have to.. but NSAssert1, NSAssert2 etc are the legacy asserts from before NSAssert could handle any number of arguments (@dasblinkenlight is wrong - NSAssert1 isn't a variadic function). Would you prefer NSLog1(), NSLog2(), etc.. — hooleyhoop, Aug 03 '12 at 13:32

score 0 · Answer 1 · answered Aug 02 '12 at 12:50

0

Use

NSAssert1(err == noErr, @"Error initializing unit: %hd", err);

or

NSAssert1(err == noErr, @"Error initializing unit: %ld", (long)err);

instead.

answered Aug 02 '12 at 12:50

score 0 · Accepted Answer · answered Aug 02 '12 at 12:51

Since NSAssert1 is a variadic function implemented within the constraints of C, it cannot perform the cast internally: it is not possible to find the type of an argument passed in the vararg section of the function. Objective-C compiler will perform the default type promotions before making the call, but the short is converted to int, not to long by default. That is why you should either insert an explicit cast, or use a different format specifier.

Wrong argument type for NSAssert1

2 Answers2