Re: [FFmpeg-devel] [PATCH]lavc/hevc_ps: Fix offset for yuv422 and yuv444

2016-03-09 Thread Carl Eugen Hoyos
Ronald S. Bultje gmail.com> writes: > > > That fixes 444, but not 422. For 422 (idc_fmt=2), horiz_mult (for l/r) > > > should be 2, but vert_mult (for t/b) should be 1. > > > > New patch attached. > > lgtm (I think). Patch applied. Thank you for your help, Carl Eugen _

Re: [FFmpeg-devel] [PATCH]lavc/hevc_ps: Fix offset for yuv422 and yuv444

2016-03-08 Thread Ronald S. Bultje
Hi, On Tue, Mar 8, 2016 at 2:03 PM, Carl Eugen Hoyos wrote: > On Tuesday 08 March 2016 07:26:31 pm Ronald S. Bultje wrote: > > On Tue, Mar 8, 2016 at 12:56 PM, Carl Eugen Hoyos wrote: > > > Attached patch fixes ticket #4980 for me. > > > Debugged by Kurosu and Hendrik. > > > > That fixes 444, bu

Re: [FFmpeg-devel] [PATCH]lavc/hevc_ps: Fix offset for yuv422 and yuv444

2016-03-08 Thread Carl Eugen Hoyos
On Tuesday 08 March 2016 07:26:31 pm Ronald S. Bultje wrote: > On Tue, Mar 8, 2016 at 12:56 PM, Carl Eugen Hoyos wrote: > > Attached patch fixes ticket #4980 for me. > > Debugged by Kurosu and Hendrik. > > That fixes 444, but not 422. For 422 (idc_fmt=2), horiz_mult (for l/r) > should be 2, but ver

Re: [FFmpeg-devel] [PATCH]lavc/hevc_ps: Fix offset for yuv422 and yuv444

2016-03-08 Thread Ronald S. Bultje
Hi, On Tue, Mar 8, 2016 at 12:56 PM, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #4980 for me. > Debugged by Kurosu and Hendrik. That fixes 444, but not 422. For 422 (idc_fmt=2), horiz_mult (for l/r) should be 2, but vert_mult (for t/b) should be 1. Ronald __

[FFmpeg-devel] [PATCH]lavc/hevc_ps: Fix offset for yuv422 and yuv444

2016-03-08 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #4980 for me. Debugged by Kurosu and Hendrik. Please comment, Carl Eugen diff --git a/libavcodec/hevc_ps.c b/libavcodec/hevc_ps.c index 64d6e2f..b4e5c5b 100644 --- a/libavcodec/hevc_ps.c +++ b/libavcodec/hevc_ps.c @@ -856,11 +856,11 @@ int ff_hevc_parse_sps(HEVCSPS