D6939: sidedata: apply basic but tight security around exchange

martinvonz (Martin von Zweigbergk) phabricator at mercurial-scm.org
Fri Oct 4 23:59:34 EDT 2019


martinvonz added inline comments.

INLINE COMMENTS

> martinvonz wrote in bundle2.py:1840-1841
> Need `b''` prefix, I think (also pick one type of quotes maybe? but maybe Black will soon change this anyway...)

Never mind... I've just gotten used to looking at extension code where the source transformer is not run. It is run on this code, of course, so this is correct as is.

REPOSITORY
  rHG Mercurial

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

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

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


More information about the Mercurial-devel mailing list