[MPlayer-dev-eng] Should there be movie_aspect and stricter prescaling checks?
ib at wupperonline.de
Wed Aug 7 17:26:33 CEST 2013
Reimar Döffinger wrote on Wed, 07 Aug 2013 08:59:47 +0200:
> "Ingo Brückl" <ib at wupperonline.de> wrote:
>>Option "aspect" (variable movie_aspect) is restricted to range 0.1..10.0
>>Command switch_ratio allows any value for movie_aspect in command.c and
>>there is already prescaling for sh->aspect > 0.01 in vd.c.
>>Should there be checks for the above range and prescaling only start if
>>greater or equal 0.1, or is it ok for the switch_ratio command to leave
>>the range of the aspect option? Should the range start with 0.01?
> I think it is reasonable for the command line to be more restricted.
I agree. My point was that all this should be consistent. It's kinda hard to
understand why option aspect starts from 0.1 and prescaling starts if > 0.01
while it is allowed to set sh->aspect to a negative value > -1.0 and < 0
(which doesn't affect scaling, but gives wrong aspect information in sh).
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 1665 bytes
Desc: not available
More information about the MPlayer-dev-eng