[FFmpeg-devel] [PATCH]Add Dirac support to ffmpeg via libdirac_* and Schroedinger libraries]
Fri Apr 11 09:01:04 CEST 2008
On Fri, Apr 11, 2008 at 12:12:11PM +1000, Anuradha Suraparaju wrote:
> On Thu, 2008-04-10 at 11:44 +0200, Diego Biurrun wrote:
> > > + int num_formats = sizeof(ffmpeg_dirac_pixel_format_map)/sizeof(ffmpeg_dirac_pixel_format_map);
> > Breaking long lines such as this one would be a plus.
> Is there a common convention that is followed everywhere in ffmpeg
> regarding long lines? I usually like to keep connected info on one line
> even though it exceeds 80 chars because it make "grep"ping for code
It is not followed everywhere - after all it has not been so long that
we enforce such things in reviews - but it is generally preferred to
keep lines below 80 characters where sensible.
The example above could be written as
int num_formats = sizeof(ffmpeg_dirac_pixel_format_map) /
which is much more readable IMO. The spaces around the / operator also
More information about the ffmpeg-devel