D7716: rust-discovery: partial switch to typestate pattern
gracinet (Georges Racinet)
phabricator at mercurial-scm.org
Fri Dec 27 17:25:35 EST 2019
gracinet added a comment.
@martinvonz yes, I've thinked of it after I made this one, but somehow I thought this was enough for a first try.
Yes, we could have `PartialDiscovery<Start>`, and `PartialDiscovery<Computed>`, with `Start` being a pure marker, and `Computed` actually holding the undecided set and the children cache (anticipating on a later changeset), that would be more elegant and it would better organize the boiler plate.
Don't hesitate if you come up with better namings.
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST ACTION
https://phab.mercurial-scm.org/D7716/new/
REVISION DETAIL
https://phab.mercurial-scm.org/D7716
To: gracinet, #hg-reviewers, martinvonz
Cc: martinvonz, Alphare, durin42, kevincox, mercurial-devel
More information about the Mercurial-devel
mailing list