[MPlayer-cvslog] r36391 - in trunk: DOCS/tech/slave.txt command.c libmpcodecs/vd.c libmpdemux/s

[Ingo Brückl]

Reimar Döffinger wrote on Fri, 02 Aug 2013 09:34:39 +0200:

> ib <subversion at mplayerhq.hu> wrote:
>>Author: ib
>>Date: Thu Aug  1 23:18:14 2013
>>New Revision: 36391
>>
>>Log:
>>Handle special argument -1 to switch_ratio as intended.
>>
>>Reset to the original aspect ratio that would have been used for
>>the very first rescaling rather than to the display size ratio.
>>
>>This will now handle anamorphic videos correctly as well.

> Maybe I miss something, but this doesn't look to me like it will handle
> files with aspect changes properly?

Yes, you're right.

It would be easier if sh->original_aspect had been used in the mpcodecs and
mpdemux, and sh->aspect would have been the current, calculated value only.
(But maybe that would have caused other problems.)

I wanted to do a "simple" modification only, so I came up with the idea to
change libmpcodecs/vd.c, but obviously underestimated the complexity of the
problem.

Anyway, this patch should handle files with aspect changes, too.

(Actually two patches, because the sh->original_aspect -1.0 thing was already
a bug in r36391.)

Ingo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: original_aspect.patch
Type: text/x-diff
Size: 2049 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/attachments/20130802/34e00abd/attachment.bin>