On 21/10/13 15:14, Luca Barbato wrote: > On 21/10/13 10:51, Diego Biurrun wrote: >> On Sun, Oct 20, 2013 at 07:20:35PM +0200, Luca Barbato wrote: >>> --- a/libavcodec/ffv1dec.c >>> +++ b/libavcodec/ffv1dec.c >>> @@ -25,17 +25,16 @@ >>> >>> +#include "libavutil/imgutils.h" >>> #include "libavutil/pixdesc.h" >>> #include "libavutil/crc.h" >>> #include "libavutil/opt.h" >>> -#include "libavutil/imgutils.h" >>> -#include "avcodec.h" >>> +#include "rangecoder.h" >>> #include "internal.h" >>> #include "get_bits.h" >>> -#include "put_bits.h" >>> -#include "rangecoder.h" >>> -#include "golomb.h" >>> +#include "avcodec.h" >>> #include "mathops.h" >>> +#include "golomb.h" >>> #include "ffv1.h" >> >> This seems pretty random and does not follow the standard we recently >> agreed on. >> > > It follows https://wiki.libav.org/HeaderOrdering >
Updated the document so it mentions that alphabetical ordering within blocks is preferred. lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
