[FFmpeg-devel] [PATCH] Pass pts values through non-delay encoders by default.

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sun Jan 22 18:06:07 CET 2012


On Sun, Jan 22, 2012 at 05:16:12PM +0100, Reimar Döffinger wrote:
> On Sun, Jan 22, 2012 at 02:39:52PM +0100, Reimar Döffinger wrote:
> > Avoids having to duplicate the code for trivial, non-reordering
> > encoders.
> > Completely and utterly breaks almost all H.264 conformance tests,
> > sometimes just making the pts start with negative values, sometimes
> > only giving 2 instead of 20 decoded frames or failing with not
> > correctly ordered pts.
> 
> Actually, all the failures where more than time stamp changes
> are short output, and that is due to non-montone time stamps
> with field interlacing.

I have an update version of the patch that also updates the references,
but it is 600 kB large, so I don't intend to send it.
However here is the list of H.264 conformance samples that we will no
longer be able to test properly after this change since they produce
completely nonsense pts values (which they did before, too, of course, it
just did not matter because they were thrown away):
+            cabac_mot_fld0_full                                         \
+            cafi1_sva_c                                                 \
+            cabref3_sand_d                                              \
+            cavlc_mot_fld0_full_b                                       \
+            cvfi2_sony_h                                                \
+            cvfi2_sva_c                                                 \
+            cvmp_mot_fld_l30_b                                          \
+            cvmp_mot_frm_l31_b                                          \
+            cvnlfi2_sony_h                                              \
+            frext-bcrm_freh10                                           \
+            frext-brcm_freh11                                           \
+            frext-brcm_freh4                                            \
+            frext-freh6                                                 \
+            frext-freh7_b                                               \
+            frext-hpcafl_bcrm_c                                         \
+            frext-hpcaflnl_bcrm_c                                       \
+            frext-hpcvfl_bcrm_a                                         \
+            frext-hpcvflnl_bcrm_a                                       \
+            mr8_bt_b                                                    \
+            sharp_mp_field_3_b                                          \


More information about the ffmpeg-devel mailing list