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