[FFmpeg-devel] [PATCH] h264 ticks and time base

[Michael Niedermayer]

On Wed, Jun 17, 2009 at 04:58:34PM -0700, Baptiste Coudurier wrote:
> Michael Niedermayer wrote:
> > On Sun, Jun 14, 2009 at 10:42:11PM -0700, Baptiste Coudurier wrote:
> >> Hi,
> >>
> >> 1) Set avctx time_base in h264 parser.
> >> 2) Set avctx ticks_per_frame in h264 decoder.
> >>
> >> I don't get this code in decode_init in h264.c
> >>
> >>     reset_sei(h);
> >>     if(avctx->codec_id == CODEC_ID_H264){
> >>         if(avctx->ticks_per_frame == 1){
> >>             s->avctx->time_base.den *=2;
> >>         }
> >>         avctx->ticks_per_frame = 2;
> >>     }
> >>     return 0;
> >>
> >> avctx->ticks_per_frame is not set before calling the decoder
> >> in any case. Should that be removed ?
> > 
> > what do you mean by "that" ?
> 
> the if(avctx->ticks_per_frame == 1)

i think it was needed because the codec might be ininted then closed and
then ininted again, but i may misremember

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In a rich man's house there is no place to spit but his face.
-- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090618/003c077b/attachment.pgp>