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

zhentan feng spyfeng at gmail.com
Sat Aug 21 12:56:45 CEST 2010


Hi

On Sat, Aug 21, 2010 at 6:37 PM, Diego Biurrun <diego at biurrun.de> wrote:

> 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.
>
> yes, thanks for reminding.
I have already fixed according to stefano's comments.

zhentan
-- 
Best wishes~


More information about the FFmpeg-soc mailing list