Alphare added a comment. Alphare marked an inline comment as done.
In D7928#119428 <https://phab.mercurial-scm.org/D7928#119428>, @marmoute wrote: > There is a lot going on in this changeset. I am counting at least: > > - new `BadType/BadMatch` structs, > - `StatusResult` → `DirstateStatus` change, > - new `StatusOptions` struct > - new `traverse_worker` function. > > Can we get the smaller cleanup as reparated chagneset before hand? It would make the thing clearer in my opinion. I don't think the `BadType/BadMatch` struct warrant a new changeset. I'll split into 3 then. REPOSITORY rHG Mercurial CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7928/new/ REVISION DETAIL https://phab.mercurial-scm.org/D7928 To: Alphare, #hg-reviewers, marmoute Cc: marmoute, durin42, kevincox, mercurial-devel _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
