[Ffmpeg-devel] [PATCH] simplify libavformat/allformats.c

Måns Rullgård mru
Sun Nov 26 17:14:18 CET 2006


Aurelien Jacobs said:
> Hi,
>
> Currently allformats.c contains more #ifdef lines than actual code.
> The attached patch highly simplify this. It almost divides by 3 the
> number of lines in this file.
> I also took this oportunity to reorder lines alphabetically.
> Ok to apply ?

Funny... I was just thinking something along the same lines.  I have
two comments:

- The reordering should be a separate patch.
- I'm not sure what I think of the combined muxer/demuxer lines.
  It saves a few lines, but having them separate keeps things
  clearer IMHO.

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




More information about the ffmpeg-devel mailing list