D6953: sidedatacopies: read rename information from sidedata

marmoute (Pierre-Yves David) phabricator at mercurial-scm.org
Wed Oct 9 17:56:17 EDT 2019


marmoute added inline comments.

INLINE COMMENTS

> martinvonz wrote in changelog.py:369-371
> Actually, both `decodefileindices()` and `decodecopies()` seem to properly handle the "empty input" case. So I think you can revert this change. As I suggested earlier, it may be best as a separate patch anyway since it seems unrelated to this patch (although maybe it is required by, if I'm still misunderstanding and it really is a bug). Sorry I didn't notice this earlier.

On this side of the discussion, confusion increases I want `None` value to be preserved (if not relevant now, if will quickly become so). As far as I understand, `decodefileindices` does not do this.

REPOSITORY
  rHG Mercurial

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

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

To: marmoute, #hg-reviewers
Cc: martinvonz, mercurial-devel


More information about the Mercurial-devel mailing list