[PATCH 1 of 5] perf: don't clobber startrev variable

Yuya Nishihara yuya at tcha.org
Tue May 9 08:58:50 EDT 2017


On Sun, 7 May 2017 03:09:50 +0530, Pulkit Goyal wrote:
> On Sat, May 6, 2017 at 11:51 PM, Gregory Szorc <gregory.szorc at gmail.com>
> wrote:
> 
> > # HG changeset patch
> > # User Gregory Szorc <gregory.szorc at gmail.com>
> > # Date 1494093578 25200
> > #      Sat May 06 10:59:38 2017 -0700
> > # Node ID 7db6c2fc82c17780d28faeee73a8d776063fbb64
> > # Parent  31f42e683321f225eb9c306f8d4b3a9e8d82a1da
> > perf: don't clobber startrev variable
> >
> > Previously, the "startrev" argument would be ignored due to
> > "startrev = 0" in the benchmark function. This meant that
> > `hg perfrevlog` always started at revision 0.
> >
> > Rename the local variable to "endrev" so the variable does the
> > ​ ​
> > right thing
> > ​.
> >
> ​                                               ^^^^^^^​
> ​I guess you mean "beginrev" here.​

Indeed. Fixed and rebased the subsequent patches.


More information about the Mercurial-devel mailing list