On Wed, Jun 29, 2011 at 07:35:30AM +0000, Gil Pedersen wrote: > On 29/06/2011, at 01.54, Diego Biurrun wrote: > > > On Tue, Jun 28, 2011 at 10:21:07AM +0200, Jindřich Makovička wrote: > >> On Thu, Jun 23, 2011 at 09:33, Tomas Härdin <[email protected]> > >> wrote: > >>> Jindřich Makovička skrev 2011-06-21 18:52: > >>>> > >>>> I am attaching another, hopefully cleaner, version of the Random > >>>> Access indicator patch for mpeg-ts muxer. > >>>> > >>>> Random Access indicator can be used to mark keyframes, and is required > >>>> by some DVB recorders to be able to index and replay the recorded > >>>> stream. > >>>> > >>>> From 7c6a9d0618e045af2026218aff92b400e7b8ee0f Mon Sep 17 00:00:00 2001 > >>>> From: Jindrich Makovicka <[email protected]> > >>>> Date: Tue, 21 Jun 2011 18:50:58 +0200 > >>>> Subject: [PATCH] set Random Access indicator on keyframe start packets > >>>> > >>>> --- > >>>> libavformat/mpegtsenc.c | 67 > >>>> +++++++++++++++++++++++++++++++++++++++------- > >>>> 1 files changed, 56 insertions(+), 11 deletions(-) > >>> > >>> Looks OK AFAICT - I don't have the spec on hand to double-check the > >>> details. > >> > >> ping? > > > > Mans, can you confirm this is good to push? > > It most likely breaks fate.
It does indeed - Jindrich, this needs to be addressed... biurrun@passion:~/src/priv/libav $ make fate-lavf-ts TEST acodec-aref TEST vsynth1-vref TEST vsynth2-vref TEST lavf-ts --- ./tests/ref/lavf/ts 2011-06-21 22:15:41.490730387 +0200 +++ tests/data/fate/lavf-ts 2011-06-29 12:05:41.926902303 +0200 @@ -1,3 +1,3 @@ -d260ac0534ff2e26b44b5192fd4fdc21 *./tests/data/lavf/lavf.ts +9708011dd80212a0041a96da878122c2 *./tests/data/lavf/lavf.ts 406644 ./tests/data/lavf/lavf.ts ./tests/data/lavf/lavf.ts CRC=0x133216c1 make: *** [fate-lavf-ts] Error 1 Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
