On Thu, Nov 13, 2014 at 9:43 PM, Anton Khirnov <[email protected]> wrote: > Quoting Vittorio Giovara (2014-11-05 15:43:44) >> On Wed, Nov 5, 2014 at 6:58 AM, Anton Khirnov <[email protected]> wrote: >> > Quoting Vittorio Giovara (2014-11-04 15:36:56) >> >> CC: [email protected] >> >> Bug-Id: CID 1238835 >> >> --- >> >> libavcodec/mpegaudio_tablegen.h | 4 ++-- >> >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> >> >> diff --git a/libavcodec/mpegaudio_tablegen.h >> >> b/libavcodec/mpegaudio_tablegen.h >> >> index 8a3e51a..110d4f2 100644 >> >> --- a/libavcodec/mpegaudio_tablegen.h >> >> +++ b/libavcodec/mpegaudio_tablegen.h >> >> @@ -44,9 +44,9 @@ static void mpegaudio_tableinit(void) >> >> { >> >> int i, value, exponent; >> >> for (i = 1; i < TABLE_4_3_SIZE; i++) { >> >> - double value = i / 4; >> >> double f, fm; >> >> int e, m; >> >> + value = i / 4; >> > >> > Eh? The code now does something completely different. >> >> Umh in what sense? > > Currently, value is double. After your patch it will be an int.
it is double, but then it cast to double again, so it make me wonder that the original intent was to use the integer version of that variable. -- Vittorio _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
