[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