[PATCH] revset: make id() to not abort on an unknown 40-byte string

Yuya Nishihara yuya at tcha.org
Mon Apr 13 10:33:55 CDT 2015


On Mon, 13 Apr 2015 14:39:07 +0000, Martin von Zweigbergk wrote:
> Would "if len(n) == 40 and n in repo:" be better?

"n in repo" just does "try repo[n] except RepoLookupError", so I'd rather do
"try rn = repo[n].rev() except ...".

It would be nice if revlog or changelog had the function to look up node
only by full or partial nodeid, but I'm not sure how it should be implemented.
I think foozy had thoughts around it.

Regards,


More information about the Mercurial-devel mailing list