[MPlayer-dev-eng] [PATCH] Fix vf_expand aspect when d_width != width

Oded Shimon ods15 at ods15.dyndns.org
Mon Dec 5 20:09:31 CET 2005


On Sun, Dec 04, 2005 at 11:38:01PM -0500, Jason Tackaberry wrote:
> The attached patch fixes vf_expand so that when specifying an aspect
> ratio, it accounts for hardware scaling (or, more accurately, when
> d_width != width) so that the final display resolution is the
> user-specified aspect ratio.
> 
> For example, I have a movie with a frame size of 704x368 and an aspect
> ratio of 2.271, so that the display size would be 836x368.  Without this
> patch, specifying -vf expand=:::::4/3 would cause a frame size of
> 704x528 and a display size of 836x528 (because 704/(4/3) = 528), which
> is not the aspect given (1.583 != 1.333).  With this patch, the expand
> height takes into account the display aspect, so the expanded frame size
> is 704x627 and display size of 836x627 (which is 4/3).

Committed. sorta. actually what I committed is nothing like the patch you 
sent, because your patch was broken :) but the general idea was preserved.

- ods15




More information about the MPlayer-dev-eng mailing list