Re: [FFmpeg-devel] [PATCH 2/3] avcodec/aacenc_is: Assert that minthr is not 0.0, this would lead to division by 0 later

On Mon, May 16, 2016 at 4:10 PM, Michael Niedermayer wrote: > yes, > > with the patch fate fails: > > Test aac-pred-encode failed. Look at tests/data/fate/aac-pred-encode.err for > details. > make: *** [fate-aac-pred-encode] Error 134 > > >> >> A threshold of 0 would in

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/aacenc_is: Assert that minthr is not 0.0, this would lead to division by 0 later

On Mon, May 16, 2016 at 12:26 PM, Kieran Kunhya wrote: >> Testcase is fate-aac-pred-encode >> >> Signed-off-by: Michael Niedermayer >> --- >> libavcodec/aacenc_is.c |3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/libavcodec/aacenc_is.c

[FFmpeg-devel] [PATCH 2/3] avcodec/aacenc_is: Assert that minthr is not 0.0, this would lead to division by 0 later

Testcase is fate-aac-pred-encode Signed-off-by: Michael Niedermayer --- libavcodec/aacenc_is.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/aacenc_is.c b/libavcodec/aacenc_is.c index 473897b..e5cfa14 100644 --- a/libavcodec/aacenc_is.c +++