[FFmpeg-devel] [PATCH 01/11] ffmpeg: use log10 instead of log()/log(10)

wm4 nfxjfg at googlemail.com
Thu Oct 29 12:36:32 CET 2015


On Thu, 29 Oct 2015 07:27:20 -0400
Ganesh Ajjanagadde <gajjanag at mit.edu> wrote:

> 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.

Not going to play this superficially-review-mass-patches-with-little-to-
none-benefit-just-so-a-single-developer-doesn't-get-annoyed-and-then-deal-
with-regression-fallout game.

You're giving the ML more patches to review than it can deal with it, so
my reaction is to flat out reject such patch-spam for things which seem to
have little benefit and are not 100% trivial.

(This reminds me of mass cosmetics from Libav...)


More information about the ffmpeg-devel mailing list