[FFmpeg-cvslog] r22010 - trunk/ffmpeg.c

[Alexander Strange]

On Feb 23, 2010, at 2:06 PM, michael wrote:

> Author: michael
> Date: Tue Feb 23 23:06:36 2010
> New Revision: 22010
> 
> Log:
> Set ist->pts to something that isnt guranteed to entangle itself with stream copying b frames.

This breaks non-streamcopy, some files have their first frame given negative pts and thus dropped.

http://astrange.ithinksw.net/ffmpeg/5cm.h264
> ./ffmpeg_g -i 5cm.h264 -an -f framecrc -y /dev/null
...
frame= 1088 fps= 55 q=0.0 Lsize=       0kB time=45.33 bitrate=   0.0kbits/s dup=0 drop=1    

I would attach a patch but I can't quite tell what this was supposed to do.

> Modified:
>   trunk/ffmpeg.c
> 
> Modified: trunk/ffmpeg.c
> ==============================================================================
> --- trunk/ffmpeg.c	Tue Feb 23 22:03:25 2010	(r22009)
> +++ trunk/ffmpeg.c	Tue Feb 23 23:06:36 2010	(r22010)
> @@ -2090,8 +2090,10 @@ static int av_encode(AVFormatContext **o
> 
>     /* init pts */
>     for(i=0;i<nb_istreams;i++) {
> +        AVStream *st;
>         ist = ist_table[i];
> -        ist->pts = 0;
> +        st= ist->st;
> +        ist->pts = st->avg_frame_rate.num ? - st->codec->has_b_frames*AV_TIME_BASE / av_q2d(st->avg_frame_rate) : 0;
>         ist->next_pts = AV_NOPTS_VALUE;
>         ist->is_start = 1;
>     }
> _______________________________________________
> ffmpeg-cvslog mailing list
> ffmpeg-cvslog at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-cvslog