On Fri Feb 06 2015 at 7:55:31 PM Timothy Gu <[email protected]> wrote:
> On Fri Feb 06 2015 at 7:44:23 PM Michael Niedermayer <[email protected]> > wrote: > >> On Fri, Feb 06, 2015 at 07:32:56AM +0000, Timothy Gu wrote: >> > On Thu Feb 05 2015 at 11:07:01 PM Timothy Gu <[email protected]> >> wrote: >> > >> > > (num_quant_units - 1) is later used as an index to >> atrac3p_qu_to_subband, >> > > which only has 32 elements (i.e. maximum of num_quant_units is 32). >> > > --- >> > > libavcodec/atrac3plus.c | 2 +- >> > > 1 file changed, 1 insertion(+), 1 deletion(-) >> > > >> > >> > Note that this doesn't actually fix any problem else than a GCC warning. >> >> when i read the commit message it sounded like it fixes a serious >> bug ... >> also the exact warning which is fixed could be mentioned in the >> commit message >> > > Yeah I've been meant to add my second mail to the commit message, but > haven't found the time to use Git yet. > Actually I misread the warning message (GCC is smarter than I thought). New patch incoming. Timothy _______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
