[FFmpeg-soc] [FFmpeg-devel] [Patch]GSoC 2008 qualification task TS Muxer

[zhentan feng]

Hi

2008/4/2, zhentan feng <spyfeng at gmail.com>:
> Hi
>
>  2008/4/2, Baptiste Coudurier <baptiste.coudurier at smartjog.com>:
>
> > On Wed, Apr 02, 2008 at 09:11:25PM +0800, zhentan feng wrote:
>  >  > [...]
>  >  >
>  >  > Only in ../socforpatch/: ffmpeg_svn.patch
>  >  > diff -ur ../socforpatch/mpegenc.c 4.2/mpegenc.c
>  >  > --- ../socforpatch/mpegenc.c  2008-04-02 17:50:09.000000000 +0800
>  >  > +++ 4.2/mpegenc.c     2008-03-31 22:49:36.000000000 +0800
>  >  > @@ -303,6 +303,16 @@
>  >
>  > >              goto fail;
>  >  >          st->priv_data = stream;
>  >  >
>  >
>  > > +        /*set PESStream format*/
>  >  > +        if(s->is_dvd){
>  >  > +           stream->format = PES_FMT_DVD;
>  >  > +        }else if(s->is_svcd){
>  >  > +           stream->format = PES_FMT_SVCD;
>  >  > +        }else if(s->is_mpeg2){
>  >  > +           stream->format = PES_FMT_MPEG2;
>  >  > +        }else{
>  >  > +           stream->format = PES_FMT_VCD;
>  >  > +        }
>  >
>  >  Default case is not VCD !
>
>
> fixed.
>
> >
>  >  > [...]
>  >
>  > >
>  >  > +
>  >  > +        if (*pts != AV_NOPTS_VALUE) {
>  >
>  > > +            if (*dts != *pts)
>  >  > +                *header_len += 5 + 5;
>  >  > +            else
>  >  > +                *header_len += 5;
>  >  > +        } else if(!(stream->format & PES_FMT_MPEG2)){
>  >  > +            (*header_len)++;
>  >  > +        }
>  >
>  >  Try to follow the codeing style here, space after if and before (
>
>
> fixed.
>
>  The new patch names "ff_pes_cal_header_4-2_2.diff" attached as below.
>
The new patch names "ff_pes_cal_header_4-3.diff" which gets rid of the
useless parenthesis.

yours
zhentan feng

-- 
Best wishes~