2008/4/3, zhentan feng <[EMAIL PROTECTED]>: > Hi > > 2008/4/2, zhentan feng <[EMAIL PROTECTED]>: > > > Hi > > > > 2008/4/2, Baptiste Coudurier <[EMAIL PROTECTED]>: > > > > > 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~
ff_pes_cal_header_4-3.diff
Description: Binary data
_______________________________________________ FFmpeg-soc mailing list FFmpeg-soc@mplayerhq.hu https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc