Quoting Luca Barbato (2017-05-23 22:08:49)
> On 5/17/17 7:46 PM, Vittorio Giovara wrote:
> > +    av_channel_layout_uninit(dst);
> > +    return av_channel_layout_copy(dst, channel_layout);
> 
> Maybe put the uninit directly in the layout_copy so there isn't risk to
> leak memory if you forget.

I am against this. Freeing memory should be explicit.

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

Reply via email to