Patchwork no \.set in graphlog

login
register
mail settings
Submitter Pierre-Yves David
Date Oct. 15, 2014, 7:59 p.m.
Message ID <eac1ab32c470c664709c.1413403196@marginatus.alto.octopoid.net>
Download mbox | patch
Permalink /patch/6286/
State Superseded
Headers show

Comments

Pierre-Yves David - Oct. 15, 2014, 7:59 p.m.
# HG changeset patch
# User Pierre-Yves David <pierre-yves.david@fb.com>
# Date 1412761431 25200
#      Wed Oct 08 02:43:51 2014 -0700
# Node ID eac1ab32c470c664709c5db6c1ddcd6bc5084034
# Parent  75d0edb68b417964110e3fcd69187c867f31a119
no \.set in graphlog
Augie Fackler - Oct. 15, 2014, 8:11 p.m.
On Wed, Oct 15, 2014 at 12:59:56PM -0700, Pierre-Yves David wrote:
> # HG changeset patch
> # User Pierre-Yves David <pierre-yves.david@fb.com>
> # Date 1412761431 25200
> #      Wed Oct 08 02:43:51 2014 -0700
> # Node ID eac1ab32c470c664709c5db6c1ddcd6bc5084034
> # Parent  75d0edb68b417964110e3fcd69187c867f31a119
> no \.set in graphlog

Can you greatly expand this log message and send again?

>
> diff --git a/mercurial/graphmod.py b/mercurial/graphmod.py
> --- a/mercurial/graphmod.py
> +++ b/mercurial/graphmod.py
> @@ -35,15 +35,14 @@ def dagwalker(repo, revs):
>
>      cl = repo.changelog
>      lowestrev = revs.min()
>      gpcache = {}
>
> -    knownrevs = revs
>      for rev in revs:
>          ctx = repo[rev]
>          parents = sorted(set([p.rev() for p in ctx.parents()
> -                              if p.rev() in knownrevs]))
> +                              if p.rev() in revs]))
>          mpars = [p.rev() for p in ctx.parents() if
>                   p.rev() != nullrev and p.rev() not in parents]
>
>          for mpar in mpars:
>              gp = gpcache.get(mpar)
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel@selenic.com
> http://selenic.com/mailman/listinfo/mercurial-devel
Pierre-Yves David - Oct. 15, 2014, 8:12 p.m.
On 10/15/2014 01:11 PM, Augie Fackler wrote:
> On Wed, Oct 15, 2014 at 12:59:56PM -0700, Pierre-Yves David wrote:
>> # HG changeset patch
>> # User Pierre-Yves David <pierre-yves.david@fb.com>
>> # Date 1412761431 25200
>> #      Wed Oct 08 02:43:51 2014 -0700
>> # Node ID eac1ab32c470c664709c5db6c1ddcd6bc5084034
>> # Parent  75d0edb68b417964110e3fcd69187c867f31a119
>> no \.set in graphlog
>
> Can you greatly expand this log message and send again?

It is supposed to be greatly expanded, not sure what happened.

Patch

diff --git a/mercurial/graphmod.py b/mercurial/graphmod.py
--- a/mercurial/graphmod.py
+++ b/mercurial/graphmod.py
@@ -35,15 +35,14 @@  def dagwalker(repo, revs):
 
     cl = repo.changelog
     lowestrev = revs.min()
     gpcache = {}
 
-    knownrevs = revs
     for rev in revs:
         ctx = repo[rev]
         parents = sorted(set([p.rev() for p in ctx.parents()
-                              if p.rev() in knownrevs]))
+                              if p.rev() in revs]))
         mpars = [p.rev() for p in ctx.parents() if
                  p.rev() != nullrev and p.rev() not in parents]
 
         for mpar in mpars:
             gp = gpcache.get(mpar)