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

INLINE COMMENTS

> dirs_multiset.rs:86
> +                    subpath.as_bytes(),
> +                );
> +                let second_slash_index = offset.unwrap() + subpath.len() - 1;

It appears to me that `find_dirs` always returns prefixes. Why is this offset 
required?

REPOSITORY
  rHG Mercurial

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

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

To: Alphare, #hg-reviewers, kevincox
Cc: durin42, kevincox, mercurial-devel
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to