D7794: rust-nodemap: generic NodeTreeVisitor
kevincox (Kevin Cox)
phabricator at mercurial-scm.org
Thu Jan 16 05:16:47 EST 2020
This revision now requires changes to proceed.
kevincox added inline comments.
kevincox requested changes to this revision.
INLINE COMMENTS
> nodemap.rs:254
> +impl<'n, 'p> Iterator for NodeTreeVisitor<'n, 'p> {
> + type Item = (bool, usize, u8, Option<Revision>);
> +
I would prefer making this a struct so that you can name the fields. You can still pattern match the fields if you want to.
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST ACTION
https://phab.mercurial-scm.org/D7794/new/
REVISION DETAIL
https://phab.mercurial-scm.org/D7794
To: gracinet, #hg-reviewers, kevincox
Cc: durin42, kevincox, mercurial-devel
More information about the Mercurial-devel
mailing list