D7923: rust-matchers: add functions to get roots, dirs and parents from patterns

kevincox (Kevin Cox) phabricator at mercurial-scm.org
Fri Jan 17 11:05:57 EST 2020


This revision now requires changes to proceed.
kevincox added inline comments.
kevincox requested changes to this revision.

INLINE COMMENTS

> matchers.rs:250
> +/// This calculates the roots and directories exactly matching the patterns and
> +/// returns a tuple of (roots, dirs) for each. It does not return other
> +/// directories which may also need to be considered, like the parent

I'm confused by the "for each" since this only returns one tuple.

> matchers.rs:311
> +    ignore_patterns: &[IgnorePattern],
> +) -> PatternResult<(HashSet<HgPathBuf>, HashSet<HgPathBuf>, HashSet<HgPathBuf>)>
> +{

Please describe the return value.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D7923/new/

REVISION DETAIL
  https://phab.mercurial-scm.org/D7923

To: Alphare, #hg-reviewers, kevincox
Cc: durin42, kevincox, mercurial-devel


More information about the Mercurial-devel mailing list