On 04/11/2012 04:44 AM, Diego Biurrun wrote: > On Thu, Apr 05, 2012 at 12:09:49PM -0400, Derek Buitenhuis wrote: >> On 05/04/2012 11:12 AM, Diego Biurrun wrote: >>> This fixes the warning: >>> libavcodec/mpegaudiodec.c:1704:14: warning: variable ‘out_size’ set but not >>> used >>> --- >>> libavcodec/mpegaudiodec.c | 4 ++-- >>> 1 files changed, 2 insertions(+), 2 deletions(-) >> >> Should this perhaps be changed to add a check like in the other decode >> functions in this file? >> >> Otherwise, LGTM. > > Justin or anyone - do you have an opinion? Decide this either way please.
I don't care much about ADU really, but I agree with Derek that it should probably be consistent with the other decode functions. -Justin _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
