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

Peter Ross pross at xvid.org
Wed Nov 7 12:23:24 CET 2012


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.

-- Peter
(A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
-------------- 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/c156d708/attachment.asc>


More information about the ffmpeg-devel mailing list