martinvonz added inline comments.

INLINE COMMENTS

> files.rs:231
> +                }
> +                auditor.audit_path(path_to_hg_path_buf(name)?)?;
> +                return Ok(name.to_owned());

Related to the comment above, this seems to (generally) return an absolute 
parent directory instead of a repo-relative path. Try adding a `println!()` 
just inside the `if same` to make sure you have test coverage.

REPOSITORY
  rHG Mercurial

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

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

To: Alphare, #hg-reviewers, kevincox, martinvonz
Cc: martinvonz, 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