[FFmpeg-soc] [soc]: r5916 - mms/mmsh.c

Diego Biurrun diego at biurrun.de
Sat Aug 21 12:37:32 CEST 2010


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


More information about the FFmpeg-soc mailing list