On Tuesday, December 20, 2011, Janne Grunau <[email protected]> wrote:
> On 2011-12-20 12:44:34 -0500, Sean McGovern wrote:
>> 'context_count' is used as a positive
>> array index so declare it as unsigned,
>> and exit early if it is zero.
>
> I think that's unnecessary. It can only happen if avctx->thread_count
> is zero and I don't think the decoder will work in that case anyway.
> If it's a real problem we should fix it somewhere else.
>
> Janne

Probably true -- gcc would still bark about it inside the function itself.
I was just being preventative more than anything else.

I'll leave it to you folks on whether to drop it or not.

-- Sean McG.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to