Wouldn't it be simpler to just check if the maximum codebook was 0 after
calculating cost1 and skipping the rest of the code in the loop?

On 29 December 2015 at 08:23, Claudio Freire <klaussfre...@gmail.com> wrote:

> Fixes an assertion error reported in #2686 that happens when
> using prediction (either explicitly or implicitly by setting
> the AAC main profile), since prediction code would allow
> creating new zeroes or removing existing ones, without
> properly checking for SF delta violations.
>
> This patch forbids creating/removing zeroes, perhaps an
> overly conservative approach, but a safe one. More permissive
> and sophisticated approaches may be attempted in the future.
>
> Attached
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to