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 _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel