[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