[FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)
gajjanag at mit.edu
Thu Oct 29 12:27:20 CET 2015
On Thu, Oct 29, 2015 at 5:43 AM, wm4 <nfxjfg at googlemail.com> wrote:
> On Thu, 29 Oct 2015 00:19:59 -0400
> Ganesh Ajjanagadde <gajjanagadde at gmail.com> wrote:
>> This is more concise and conveys the intent better.
>> Furthermore, it is likely more precise as well due to lack of floating
>> point division.
>> Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
> These patches are all pretty similar. And likely tedious to check for
> correctness. So low gain, while some potential for regressions.
In the time it took you to write that comment, you could have easily
reviewed a couple. log10 is already being used in the codebase, why
not make it consistent and also make it more precise?
There is a good reason why libc has a log10 function.
Maybe you don't care about such things, but I do: it is (roughly)
analogous to using double instead of float for filters/resampling etc
- noise floor should not be increased unless there is a clear benefit.
Here there is none from using log as opposed to log10.
> NACK from me.
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
More information about the ffmpeg-devel