This is an archive of the discontinued Mercurial Phabricator instance.

Status	Author	Revision
Closed	marmoute	D6823 flagprocessors: remove flagprocessorsmixin
Closed	marmoute	D6822 flagprocessors: move _flagserrorclass attribute on revlog & co
Closed	marmoute	D6821 flagprocessors: directly duplicate the deprecated layer back into revlog
Closed	marmoute	D6820 flagprocessors: make `processflagsraw` a module level function
Closed	marmoute	D6819 flagprocessors: make `processflagsread` a module level function
Closed	marmoute	D6818 flagprocessors: make `processflagswrite` a module level function
Closed	marmoute	D6817 flagprocessors: make `_processflagsfunc` a module level function
Closed	marmoute	D6816 flagprocessors: writetransform function take side data as parameter (API)
Closed	marmoute	D6815 flagprocessors: add a `sidedata` parameters to _processflagswrite
Closed	marmoute	D6814 revlog: add a `sidedata` parameters to addrevision
Closed	marmoute	D6813 flagprocessors: have the read transform function return side data (API)
Closed	marmoute	D6812 flagprocessors: return flagdata in the main processing function
Closed	marmoute	D6811 flagprocessors: return sidedata map in `_processflagsread`
Closed	marmoute	D6810 revlog: use the new sidedata map return in the sidedata method
Closed	marmoute	D6809 revlog: return sidedata map from `_revisiondata`
Closed	marmoute	D6808 revlog: introduce a `sidedata` method
Closed	marmoute	D6807 flagprocessors: small code update to clarify parameters
Closed	marmoute	D6806 flagprocessors: _processflags
Closed	marmoute	D6805 simplestorerepo: stop using `_processflags` directly
Closed	marmoute	D6804 revlog: stop using `_processflags` directly
Closed	marmoute	D6803 flagprocessors: use _processflagsraw in easy cases
Closed	marmoute	D6802 flagprocessors: use _processflagsread in simple cases
Closed	marmoute	D6801 flagprocessors: use _processflagswrite for write operation
Closed	marmoute	D6800 flagprocessors: introduce specialized functions
Closed	marmoute	D6799 flagutil: use it in simplestorerepo
Closed	marmoute	D6798 flagutil: make the error class used by the mixin configurable
Closed	marmoute	D6797 flagutil: use the new mixin use in remotefilelog
Closed	marmoute	D6796 flagutil: introduce a flagprocessorsmixin class

Diff 16410

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)

Diff	ID	Description	Created	Lint	Unit
Base		Base
Diff 1	16410		Sep 7 2019, 5:27 AM	★	★
Diff 2	16444	rHG38c62272f46208cbe84fb029a776e5d6cc6fc4da	Sep 2 2019, 11:06 AM	★	★

This is an archive of the discontinued Mercurial Phabricator instance.

simplestorerepo: stop using `_processflags` directlyClosedPublic

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 16410

tests/simplestorerepo.py

simplestorerepo: stop using `_processflags` directly
ClosedPublic

Revision Contents
Changeset List