D1351: changegroup: use any node, not min(), in treemanifest's generatemanifests
martinvonz (Martin von Zweigbergk)
phabricator at mercurial-scm.org
Fri Nov 10 00:51:43 EST 2017
martinvonz added inline comments.
INLINE COMMENTS
> spectral wrote in changegroup.py:738
> I think that it needs to remain, makelookupmflinknode(dir) relies on it (L716). I haven't attempted to popitem and pass that to makelookupmflinknode instead, let me try that out now...
Oh, yuck. Can perhaps pass "nodes" into makelookupmflinknode() along with "dir"? It will still update it on line 721, but at least one of those unclear sideeffects go away.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D1351
To: spectral, #hg-reviewers
Cc: indygreg, martinvonz, mercurial-devel
More information about the Mercurial-devel
mailing list