Re: [PATCH 8/8] perf: add tests for tag --contains

From: Jeff King <peff@peff.net>
Date: Jun 26, 2014, 00:04 UTC
Message-ID: <20140626000405.GB25028@sigill.intra.peff.net>
In-Reply-To: <20140626000129.GA25028@sigill.intra.peff.net>

On Wed, Jun 25, 2014 at 08:01:29PM -0400, Jeff King wrote:

Show 12 quoted lines

> I get:
> 
>   Test                              ffc4b80^          origin/master             HEAD                  
>   ----------------------------------------------------------------------------------------------------
>   7000.3: contains recent/all       1.99(1.97+0.01)   0.25(0.24+0.00) -87.4%    0.27(0.26+0.00) -86.4%
>   7000.4: contains recent/v2.0.1    0.03(0.03+0.00)   0.00(0.00+0.00) -100.0%   0.03(0.02+0.00) +0.0% 
>   7000.5: contains old/all          0.90(0.89+0.00)   0.18(0.17+0.00) -80.0%    0.27(0.26+0.00) -70.0%
>   7000.6: contains old/v2.0.1       0.25(0.24+0.00)   0.03(0.03+0.00) -88.0%    0.25(0.24+0.00) +0.0% 
>   7000.7: contains ancient/all      0.82(0.80+0.01)   0.13(0.12+0.00) -84.1%    0.27(0.26+0.01) -67.1%
>   7000.8: contains ancient/v2.0.1   0.26(0.26+0.00)   0.09(0.08+0.00) -65.4%    0.25(0.24+0.00) -3.8% 
> 
> which looks about right.

Oh, hmph. The "bad" case in recent/v2.0.1 for origin/master goes away then. Because we get "lucky" again, and the depth-first one does not have to go all the way to the roots. So it's probably a better example to use the original HEAD~100, which isn't actually in v2.0.1.

-Peff