[FFmpeg-devel] FFmpeg source code is no longer C99 because of GNUism called case ranges

Måns Rullgård mans
Sat Jul 7 23:43:38 CEST 2007


Roman Shaposhnik <rvs at sun.com> writes:

> Hi
>
> On Sat, 2007-07-07 at 22:13 +0100, M?ns Rullg?rd wrote:
>> >   Well, hopefully this is the same thing Michael had in mind ;-) But we
>> > are supposed to find out soon.
>> 
>> I personally prefer to break lines after operators, like so:
>> 
>> else if(avctx->sub_id == 0x20001000 ||
>>         avctx->sub_id >= 0x201a0000 && avctx->sub_id < 0x201a0000)
>
>   That is my personal style as well. That said, I really wish to see
> guidelines like those in our docs somewhere so that submitters don't
> have to struggle with guesswork.

Most of the code in FFmpeg uses this style, so you should be fine if
you stick with it.  (And I like it better that way ;-)

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-devel mailing list