On Tue, May 23, 2017 at 4:08 PM, Luca Barbato <lu_z...@gentoo.org> wrote:
> 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.

Added locally, ok otherwise?
-- 
Vittorio
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to