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

[Michael Niedermayer]

Hi

On Mon, Mar 05, 2007 at 11:57:34PM +0100, Moritz Bunkus wrote:
> 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).

its impossible for 2 different ARs to be correct, one will cause peoples
heads to look like squished eggs :)
so IMHO the wrong one of the 2 should be corrected

besides that yes of course should the mux level get priority over codec
(unless its QT where you must do it the other way around ...)


> 
> 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.

if the bitstream contains a wrong AR its broken and its not the muxers
job to correct that, of course storing a different AR at container level
than what is stored at bitstream level is even worse
so depending on architecture you might really end up with having to
remove it at muxer level ...
in ffmpeg though the correct spot is a bitstream filter to remove or
change the AR if thats needed

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I count him braver who overcomes his desires than him who conquers his
enemies for the hardest victory is over self. -- Aristotle
-------------- 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/20070306/00ccfbe1/attachment.pgp>