On Sat, 1 Oct 2016, Josh de Kock wrote:
On 01/10/2016 17:17, Marton Balint wrote:Fixes ticket #5873. Signed-off-by: Marton Balint <c...@passwd.hu> --- ffplay.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/ffplay.c b/ffplay.c index e64909a..a7a6785 100644 --- a/ffplay.c +++ b/ffplay.c @@ -2224,7 +2224,6 @@ static int video_thread(void *arg) static int subtitle_thread(void *arg) { VideoState *is = arg; - AVCodecParameters *codecpar = is->subtitle_st->codecpar; Frame *sp; int got_subtitle; double pts; @@ -2243,8 +2242,8 @@ static int subtitle_thread(void *arg) pts = sp->sub.pts / (double)AV_TIME_BASE; sp->pts = pts; sp->serial = is->subdec.pkt_serial; - sp->width = codecpar->width; - sp->height = codecpar->height; + sp->width = is->subdec.avctx->width; + sp->height = is->subdec.avctx->height; sp->uploaded = 0; /* now we can update the picture count */Feels backwards to go to a avctx from codecpar, but if it fixes 5873 then it's fine, it can always be changed back again in the future--so LGTM.
Pushed both patches, thanks. Marton _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
