Hi On Sat, Aug 21, 2010 at 6:37 PM, Diego Biurrun <[email protected]> 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~ _______________________________________________ FFmpeg-soc mailing list [email protected] https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc
