[MPlayer-cvslog] r36401 - trunk/libmpcodecs/vd.c

[Reimar Döffinger]

On 05.08.2013, at 17:59, Ingo Brückl <ib at wupperonline.de> wrote:

> I wrote on Mon, 05 Aug 2013 15:12:35 +0200:
> 
>> Reimar Döffinger wrote on Mon, 5 Aug 2013 08:53:42 +0200:
> 
>>> On Sun, Aug 04, 2013 at 10:19:58PM +0200, ib wrote:
>>>> +    static float last_movie_aspect;
> 
>>> I don't have time to think about it right now, but could you please
>>> think of another solution?
> 
>> Actually, the whole original_aspect stuff should be realized differently.
> 
> Here you are. (Hopefully, I didn't miss a demuxer.)
> 
> This separates the original aspect from the calculated one, and the code in
> vd.c is very clear again.
> 
> Please tell, which patch you prefer.

That one seems fine to me, except that the vd_mpegpes change seems completely broken to me (will cause continuous reinits if a different aspect is forced into sh_aspect).