( )⚙ D6805 simplestorerepo: stop using `_processflags` directly

This is an archive of the discontinued Mercurial Phabricator instance.

Differential D6805

simplestorerepo: stop using `_processflags` directly
ClosedPublic

Authored by marmoute on Sep 7 2019, 5:27 AM.

Details

Reviewers
yuja
durin42
indygreg
Group Reviewers
hg-reviewers
Commits
rHG38c62272f462: simplestorerepo: stop using `_processflags` directly
Summary

We now use the specialized versions.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

marmoute created this revision.Sep 7 2019, 5:27 AM
indygreg accepted this revision.Sep 7 2019, 1:06 PM
This revision is now accepted and ready to land.Sep 7 2019, 1:06 PM
marmoute added a commit: rHG38c62272f462: simplestorerepo: stop using `_processflags` directly.Sep 7 2019, 1:08 PM
Closed by commit rHG38c62272f462: simplestorerepo: stop using `_processflags` directly (authored by marmoute). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mtests/simplestorerepo.py (6 lines)

Diff 16444

tests/simplestorerepo.py

return b'' return b''
rev = self.rev(node) rev = self.rev(node)
flags = self._flags(rev) flags = self._flags(rev)
path = b'/'.join([self._storepath, hex(node)]) path = b'/'.join([self._storepath, hex(node)])
rawtext = self._svfs.read(path) rawtext = self._svfs.read(path)
text, validatehash = self._processflags(rawtext, flags, 'read', raw=raw) if raw:
validatehash = self._processflagsraw(rawtext, flags)
text = rawtext
else:
text, validatehash = self._processflagsread(rawtext, flags)
if validatehash: if validatehash:
self.checkhash(text, node, rev=rev) self.checkhash(text, node, rev=rev)
return text return text
def rawdata(self, nodeorrev): def rawdata(self, nodeorrev):
return self.revision(raw=True) return self.revision(raw=True)