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

[Nico Sabbi]

Moritz Bunkus wrote:

>Hey,
>
>On Monday 05 March 2007 23:02, Piotr Kaczuba wrote:
>
>  
>
>>The problem with mkvmerge is that it removes the SAR from the
>>bitstream.  The author is of the opinion that the AR should only be
>>stored in the container. You can override this with "--engage
>>keep_bitstream_ar_info", however.
>>    
>>
>
>My opinion is simply that most container information is to be regarded
>as authoriative. With a generic tool that understands the container you
>can change all relevant information, even if new codecs etc are
>developped. mkvmerge removes the bitstream AR info so that there's no
>ambiguity, especially if the container AR is changed afterwards.
>
>Mosu
>  
>
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).
BTW, does the muxer patch every video stream? so does the muxer have to know
the details of every video stream it accepts?