On 5/26/17 6:03 PM, Vittorio Giovara wrote: > 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?
If you also changed av_channel_layout_from_mask, the rest of the set seems fine as well. lu _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel