[FFmpeg-devel] [PATCH 1/5] wtvenc: do not emit stream2 and DSATTRIB_TRANSPORT_PROPERTY chunks; these are not required for playback

Michael Niedermayer michaelni at gmx.at
Wed Nov 7 13:44:40 CET 2012


On Wed, Nov 07, 2012 at 10:23:24PM +1100, Peter Ross wrote:
> On Wed, Nov 07, 2012 at 11:09:37AM +0100, Michael Niedermayer wrote:
> > On Wed, Nov 07, 2012 at 04:51:51PM +1100, Peter Ross wrote:
> > > ---
> > >  libavformat/wtvenc.c |   30 +++---------------------------
> > >  tests/ref/lavf/wtv   |    2 +-
> > >  2 files changed, 4 insertions(+), 28 deletions(-)
> > 
> > This fails this:
> > 
> > ...
> > -ret: 0         st: 1 flags:1 dts: 0.459297 pts: 0.459297 pos: 206056 size:   209
> > +ret: 0         st: 1 flags:1 dts: 0.459297 pts: 0.459297 pos: 205328 size:   209
> >  ret: 0         st:-1 flags:1  ts:-0.645825
> > -ret: 0         st: 1 flags:1 dts:-0.010907 pts:-0.010907 pos:  27072 size:   208
> > +ret: 0         st: 1 flags:1 dts:-0.010907 pts:-0.010907 pos:  26344 size:   208
> > Test seek-lavf_wtv failed. Look at tests/data/fate/seek-lavf_wtv.err for details.
> > make: *** [fate-seek-lavf_wtv] Error 1
> > make: *** Waiting for unfinished jobs....
> > 
> > I can update the checksums myself as well but i dont want to mistakely
> > commit a change that has unintended sideeffects though this looks
> > expected ...
> 
> Aah. Please do. The three patches that touch tests/ref/lavf/wtv will need updating.
> I have of course tested theses changes with the Microsoft and FFmpeg implementations
> of wtv.

ok, applied


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many that live deserve death. And some that die deserve life. Can you give
it to them? Then do not be too eager to deal out death in judgement. For
even the very wise cannot see all ends. -- Gandalf
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121107/2383af25/attachment.asc>


More information about the ffmpeg-devel mailing list