D7406: scmutil: convert status data object from a tuple to an attrs (API)

durin42 (Augie Fackler) phabricator at mercurial-scm.org
Fri Nov 15 12:27:15 EST 2019


durin42 added inline comments.

INLINE COMMENTS

> martinvonz wrote in scmutil.py:81-88
> What depends on this? It looked like many of the patches earlier in the series tried to get rid of iteration.

I started out trying to get rid of the iterator interface on this, and gave up because it's just too many callsites for me to care.

I do think, long term, we should endeavor to rip this out.

REPOSITORY
  rHG Mercurial

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

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

To: durin42, #hg-reviewers, indygreg
Cc: Alphare, martinvonz, indygreg, mercurial-devel


More information about the Mercurial-devel mailing list