[MPlayer-dev-eng] [PATCH] Fix aspect in mkv

[Moritz Bunkus]

Hey,

On Monday 05 March 2007 23:53, Nico Sabbi wrote:

> it's common sense and common practice giving higher-level elements
> (mux in this case) higher priority over lower level ones (video
> bitstream) so I don't see the need to modify the video stream (it's
> rather the opposite: the original information is lost for good, hence
> the muxer is doing a damage).

That's true as well. The reason I implemented removing the AR info in
the first place was that there were servere playback issues on Windows
-- the splitter and various video decoders (ffdshow, xvid, Nero's MPEG 4
decoder) were behaving differently if there was bitstream AR that
differed from container AR, and the only way of achieving a consistent
playback behavior was by removing the bitstream AR info.

You may say that it's common sense, but try to get Nero to change their
codec ;)

> BTW, does the muxer patch every video stream? so does the muxer have
> to know the details of every video stream it accepts?

This is not part of the Matroska spec, it's only something that both
Haali's Matroska muxer and mkvmerge do, and they only do it for MPEG-4
part 2 video streams.

Mosu

-- 
If Darl McBride was in charge, he'd probably make marriage
unconstitutional too, since clearly it de-emphasizes the commercial
nature of normal human interaction, and probably is a major impediment
to the commercial growth of prostitution. - Linus Torvalds

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20070305/e591674e/attachment.pgp>