On Thu, Oct 29, 2015 at 5:47 PM, Michael Niedermayer <mich...@niedermayer.cc> wrote: > On Thu, Oct 29, 2015 at 12:20:05AM -0400, Ganesh Ajjanagadde wrote: >> This is likely more precise and conveys the intent better. >> >> Signed-off-by: Ganesh Ajjanagadde <gajjanaga...@gmail.com> >> --- >> libavfilter/avf_showspectrum.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libavfilter/avf_showspectrum.c b/libavfilter/avf_showspectrum.c >> index 936db60..7d5c438 100644 >> --- a/libavfilter/avf_showspectrum.c >> +++ b/libavfilter/avf_showspectrum.c >> @@ -386,7 +386,7 @@ static int plot_spectrum_column(AVFilterLink *inlink, >> AVFrame *insamples) >> a = cbrt(a); >> break; >> case LOG: >> - a = 1 - log(FFMAX(FFMIN(1, a), 1e-6)) / log(1e-6); // zero >> = -120dBFS >> + a = 1 + log10(FFMAX(FFMIN(1, a), 1e-6)) / 6; // zero = >> -120dBFS > > LGTM > > thx
pushed, thanks > > [...] > -- > Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB > > While the State exists there can be no freedom; when there is freedom there > will be no State. -- Vladimir Lenin > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel