On Tue, Oct 21, 2014 at 3:09 PM, Tim Walker <[email protected]> wrote: > On 21 Oct 2014, at 16:02, Vittorio Giovara <[email protected]> wrote: > >> On Tue, Oct 21, 2014 at 2:52 PM, Tim Walker <[email protected]> wrote: >>> On 21 Oct 2014, at 13:43, Vittorio Giovara <[email protected]> >>> wrote: >>> >>>> From the specifications: "The pixel count of the track >>>> (PixelWidth/PixelHeight) >>>> should be the raw amount of pixels (for example 3840x1080 for full HD side >>>> by >>>> side) and the DisplayWidth/Height in pixels should be the amount of pixels >>>> for >>>> one plane (1920x1080 for that full HD stream)," >>>> >>>> So, move the aspect ratio check in the mkv_write_stereo_mode() function >>>> and adapt the frame size appropriately when necessary. >>>> >>>> CC: [email protected] >>>> Found-by: Asan Usipov <[email protected]> >>>> --- >>>> libavformat/matroskaenc.c | 28 +++++++++++++++++++++------- >>>> 1 file changed, 21 insertions(+), 7 deletions(-) >>> >>> Is MVC unaffected by this patch? >> >> I guess so, what is the exact usecase you're referring to? > > Just making sure that if 1080p MVC is encountered (where the bitstream's > display width/height is still 1920x1080, isn't it?), the display width/height > are not set to 960x1080 or the likes ;)
No, this is set only when frame packed stereo is found (side-by-side and friends). Wait, you have a MVC decoder and didn't tell us? ;) -- Vittorio _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
