D1340: dirstate: add explicit methods for modifying dirstate
durin42 (Augie Fackler)
phabricator at mercurial-scm.org
Fri Nov 10 17:06:53 EST 2017
durin42 requested changes to this revision.
durin42 added inline comments.
This revision now requires changes to proceed.
INLINE COMMENTS
> dirstate.py:130
> def _map(self):
> - '''Return the dirstate contents as a map from filename to
> - (state, mode, size, time).'''
> + '''Returns the dirstate map.'''
> self._map = dirstatemap(self._ui, self._opener, self._root)
Why is ~all the interesting content of this docstring removed? Where is the meaning of the dirstate map documented other than this docstring?
> dirstate.py:1196
>
> class dirstatemap(object):
> def __init__(self, ui, opener, root):
Per my comment above, let's insert a patch before this one that documents in more detail the API that dirstatemap is supposed to be providing, probably in a docstring on this class. Sound reasonable?
(Then the above docstring I complained about could point to this docstring.)
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D1340
To: mbthomas, #hg-reviewers, durin42
Cc: durin42, mbolin, mercurial-devel
More information about the Mercurial-devel
mailing list