Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: Fix decoding regression due to height check

2017-07-23 Thread Michael Niedermayer
On Sat, Jul 22, 2017 at 05:14:10PM +, Kieran Kunhya wrote: > > > > LGTM. Look at Libav repo, it is checked with a_height. > > > > Thank's. > > > > Patch OK. As author of this code I would highly recommend not looking at > libav repo because they broke a ton of stuff. will be in my next git pu

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: Fix decoding regression due to height check

2017-07-22 Thread Kieran Kunhya
> > LGTM. Look at Libav repo, it is checked with a_height. > > Thank's. > Patch OK. As author of this code I would highly recommend not looking at libav repo because they broke a ton of stuff. Kieran ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: Fix decoding regression due to height check

2017-07-22 Thread Muhammad Faiz
On Fri, Jul 21, 2017 at 4:49 PM, Michael Niedermayer wrote: > From: Vodyannikov Aleksandr > > Fixes: Ticket6546 > > Regression since: 54aaadf648073149f1ac34f56cbde4e6c5aa22ef > > Signed-off-by: Michael Niedermayer > --- > libavcodec/cfhd.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)

[FFmpeg-devel] [PATCH] avcodec/cfhd: Fix decoding regression due to height check

2017-07-21 Thread Michael Niedermayer
From: Vodyannikov Aleksandr Fixes: Ticket6546 Regression since: 54aaadf648073149f1ac34f56cbde4e6c5aa22ef Signed-off-by: Michael Niedermayer --- libavcodec/cfhd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/cfhd.c b/libavcodec/cfhd.c index f78bad9ae1..5ea8f24