John Stebbins wrote:
On 08/25/2011 07:45 PM, John Stebbins wrote:
 As we discussed on IRC this morning, this makes the value of repeat_pict 
returned
 by the parsers the same as the value returned by the decoders.

 The parser was returning a value for repeat_pict that was the
 actual number of repeats + 1.

 Passes fate with the exception of 1 pre-existing failure 
(fate-h264-conformance-mr3_tandberg_b).

Heh, fate passed, but some samples I have that use pulldown didn't play
smoothly with avplay.  One line fix in compute_frame_duration.



@@ -1589,7 +1589,7 @@ static int output_packet(InputStream *ist, int ist_index,
                     }
                     ist->next_pts = ist->pts = 
guess_correct_pts(&ist->pts_ctx, picture.pkt_pts, picture.pkt_dts);
                     if (ist->st->codec->time_base.num != 0) {
-                        int ticks= ist->st->parser ? ist->st->parser->repeat_pict+1 : 
ist->st->codec->ticks_per_frame;
+                        int ticks= ist->st->parser ? 
ist->st->parser->repeat_pict+ist->st->codec->ticks_per_frame :
+                                   ist->st->codec->ticks_per_frame;

int ticks = st->st->codec->ticks_per_frame + (ist->st->parser ? 
ist->st->parser->repeat_pict : 0);

_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to