On 5/26/17 7:30 PM, Anton Khirnov wrote: > 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. >
Pretty much every use (bar maybe 2) of the two mentioned function needs an uninit in front of it. lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
