On Mon, Feb 10, 2014 at 11:43:52AM -0500, Andrew Kelley wrote:
> On Mon, Feb 10, 2014 at 7:13 AM, Diego Biurrun <[email protected]> wrote:
> > On Sat, Feb 08, 2014 at 08:59:33PM -0500, Andrew Kelley wrote:
> >> +    int chan, i, av_uninit(dindex), oindex, av_uninit(count);
> >
> > The av_uninit look fishy.  Why are they necessary?
> 
> Because they are only assigned inside the loop that iterates over each 
> channel.
> We assume that there will be at least one channel, but the compiler does not
> know that. Hence the av_uninit.

Maybe just initialize to zero then.  av_uninit is somewhat dodgy.

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

Reply via email to