Skip to content
Snippets Groups Projects
Commit 7d69fdb7 authored by Andrew Clayton's avatar Andrew Clayton Committed by Tristan Matthews
Browse files

vpx: decoder: Set i_sar_num and i_sar_den


Since commit 0bceaf96 ("vpx: increase decoder capability (refs #16836)")
made libvpx the default this has broken the default playback of VP8/9
video where the sample aspect ratio (SAR) is not equal to one. e.g

Given a video like

    vp8, yuv420p, 720x576, SAR 64:45 DAR 16:9, 25 fps

which is meant to be played back at 1024x576, will playback at 720x576
due to not passing the SAR value through.

I have many such videos as encoded from PAL DVD's. Another example is

    vp8, yuv420p, 720x576, SAR 16:15 DAR 4:3, 25 fps

this should actually playback at 768x576.

So this commit simply passes the SAR values through. Without this
passing --codec=ffmpeg or --codec=avcodec also restores previous
behaviour.

Signed-off-by: default avatarAndrew Clayton <andrew@digital-domain.net>
Signed-off-by: default avatarTristan Matthews <tmatth@videolan.org>
parent 83e21b1d
No related merge requests found
......@@ -328,6 +328,11 @@ static int OpenDecoder(vlc_object_t *p_this)
dec->fmt_out.video.i_width = dec->fmt_in.video.i_width;
dec->fmt_out.video.i_height = dec->fmt_in.video.i_height;
if (dec->fmt_in.video.i_sar_num > 0 && dec->fmt_in.video.i_sar_den > 0) {
dec->fmt_out.video.i_sar_num = dec->fmt_in.video.i_sar_num;
dec->fmt_out.video.i_sar_den = dec->fmt_in.video.i_sar_den;
}
return VLC_SUCCESS;
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment