[Commented On] D9964: rhg: Parse per-repository configuration

[Alphare (Raphaël Gomès)]

Alphare added inline comments.

INLINE COMMENTS

> SimonSapin wrote in config.rs:29
> With three vecs the precedence logic would be centralized in a method that returns `impl Iterator<Item=&ConfigLayer>`.
> 
> Which do you mean by keep it this way? With `RangeFrom<usize>`?

> With three vecs the precedence logic would be centralized in a method that returns impl Iterator<Item=&ConfigLayer>.

Right, while it's not particularly harder, I didn't feel like it was needed when adding the config logic. If you think that it'll be frequent enough to query the separate "groups" of layers then by all means, go ahead. :)

> Which do you mean by keep it this way? With RangeFrom<usize>?

I meant using a single `Vec`. I am still -1 on using `RangeFrom<usize>`.

REPOSITORY
  rHG Mercurial

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

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

To: SimonSapin, #hg-reviewers
Cc: Alphare, mercurial-patches
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mercurial-scm.org/pipermail/mercurial-patches/attachments/20210208/e4e6b386/attachment-0002.html>