From: apodtele <apodtele@gmail.com>
Date: Fri, 13 Oct 2006 13:25:09 GMT
Subject: Re: [PATCH 1/2] diff --stat: use asymptotic scaling in graph
Message-ID: <d620685f0610130625o2d5f70c5p7cb41f567093df32@mail.gmail.com>
In-Reply-To: <Pine.LNX.4.64.0610122038330.2435@xanadu.home>

Hi!

On 10/12/06, Nicolas Pitre <nico@cam.org> wrote:
> On Thu, 12 Oct 2006, A Large Angry SCM wrote:
> > Martin Waitz wrote:
> > > On Thu, Oct 12, 2006 at 03:20:09PM -0700, A Large Angry SCM wrote:
> > > > > +    if (it)
> > > > > +        return it * width / (it + width) + 1;
> > > > > +    else
> > > > > +        return 0;
> > > > No conditional needed:
> > > >
> > > >  return it * width / (it + width - 1)
> > >
> > > But then it would start scaling much earlier
> > > (for width 10: at 2 instead of 4).
> > > This is not bad per se, but different...
> > >
> >
> > OK:
> >       return (it * width + (it + width)/2)) / (it + width - 1)
> >
> > Now it's back at 4. ;-)
>
> Sure, but at this point the original conditional is probably more
> efficient.
>

Don't make me use
            return it * width / (it + width) + !!it;