On Wed, Aug 18, 2010 at 06:04:18PM +0200, spyfeng wrote: > > Log: > modify include file order. > > --- mms/mmsh.c Wed Aug 18 09:03:02 2010 (r5915) > +++ mms/mmsh.c Wed Aug 18 18:04:18 2010 (r5916) > @@ -24,11 +24,11 @@ > * Windows Media HTTP Streaming Protocol. > * http://msdn.microsoft.com/en-us/library/cc251059(PROT.10).aspx > */ > -#include "mms.h" > -#include "internal.h" > -#include "libavutil/intreadwrite.h" > #include <string.h> > +#include "libavutil/intreadwrite.h" > #include "libavutil/avstring.h" > +#include "libavutil/internal.h" > +#include "mms.h" > #include "asf.h" > #include "http.h"
For extra readability I suggest adding an empty line between system and local headers as well as between license comment and #includes. Diego _______________________________________________ FFmpeg-soc mailing list [email protected] https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc
