On Wed, Oct 28, 2015 at 9:20 PM, Ganesh Ajjanagadde <gajjanaga...@gmail.com> wrote: > This is likely more precise and conveys the intent better. > > Signed-off-by: Ganesh Ajjanagadde <gajjanaga...@gmail.com> > --- > libavfilter/af_volumedetect.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavfilter/af_volumedetect.c b/libavfilter/af_volumedetect.c > index 01f24ba..ad5fef3 100644 > --- a/libavfilter/af_volumedetect.c > +++ b/libavfilter/af_volumedetect.c > @@ -78,7 +78,7 @@ static inline double logdb(uint64_t v) > double d = v / (double)(0x8000 * 0x8000); > if (!v) > return MAX_DB; > - return log(d) * -4.3429448190325182765112891891660508229; /* -10/log(10) > */ > + return log10(d) * 10;
Wouldn't this reverse the sign of the result? > } > > static void print_stats(AVFilterContext *ctx) > -- > 2.6.2 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
