[Ffmpeg-devel] More signed fun

Måns Rullgård mru
Sun Mar 26 15:24:20 CEST 2006


Diego Biurrun <diego at biurrun.de> writes:

> On Thu, Mar 23, 2006 at 10:32:53AM +0000, M?ns Rullg?rd wrote:
>> 
>> Steve Lhomme said:
>> >
>> > Depending on how your preprocessor handle data, not using () in macros
>> > can lead to problems. Like interpreting (- - variable) (invalid C
>> > syntax)instead of -(-variable). Here is a fix.
>> 
>> - -x is perfectly valid C syntax.  If your compiler chokes on it,
>> it's broken.
>> 
>> That said, the fix is appropriate for other reasons.
>
> So will you commit it?

Actually, looking closer those parens are not needed at all.  The
macro is only used a few times just below the definition, and always
with a single number for each argument.  Those parens only add useless
clutter in this case.

-- 
M?ns Rullg?rd
mru at inprovide.com





More information about the ffmpeg-devel mailing list