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

[zhentan feng]

2008/4/3, zhentan feng <spyfeng at gmail.com>:
> 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.
>

Sorry forget to attach the patch



-- 
Best wishes~
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ff_pes_cal_header_4-3.diff
Type: application/octet-stream
Size: 12259 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-soc/attachments/20080403/a4bc368c/attachment.obj>