[vlc-devel] Re: [patch] Fixed source aspect ratio calculation

[Marian Durkovic]

> isn't it better to resort to the x and y settings defined in the 
> sequence_display_extension
> (I don't remember the names atm), rather than hardcoding those values?

That's exactly what my patch is doing at first place, but some MPEG2
encoders are broken and fill those fields with 1088 - here is the
sample from HD-1:

mpeg2dec-0.4.0 - by Michel Lespinasse <walken at zoy.org> and Aaron Holtzman
      9c SEQUENCE MPEG2 MP at HL 1920x1088 chroma 960x544 fps 25 maxBps 2061250 vbv
 999424 picture 1920x1088 display 1920x1088 pixel 136x135


And this is from correctly encoded 1080i stream:

mpeg2dec-0.4.0 - by Michel Lespinasse <walken at zoy.org> and Aaron Holtzman
      26 SEQUENCE MPEG2 MP at HL 1920x1088 chroma 960x544 fps 29.97 maxBps 2250000
vbv 999424 picture 1920x1080 display 1920x1080 pixel 1x1


	With kind regards,

		M.

-- 
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html