comparison mercurial/filelog.py @ 39861:96838b620b9c

filelog: store filename directly on revlog instance This attribute is only used by LFS. It is used by one of the revlog flag processor functions, which gets an instance of the revlog - not the file storage type. So, it makes sense to store this attribute on the revlog instead of the filelog. With this change, I'm pretty confident that LFS is no longer directly accessing file storage interface members that are revlog centric. i.e. it gets us one step closer to eliminating revlog-centric APIs from the file storage interface! Differential Revision: https://phab.mercurial-scm.org/D4715
author Gregory Szorc <gregory.szorc@gmail.com>
date Thu, 20 Sep 2018 18:07:42 -0700
parents 62a532045e71
children 32d3ed3023bb
comparison
equal deleted inserted replaced
39860:62a532045e71 39861:96838b620b9c
22 self._revlog = revlog.revlog(opener, 22 self._revlog = revlog.revlog(opener,
23 '/'.join(('data', path + '.i')), 23 '/'.join(('data', path + '.i')),
24 censorable=True) 24 censorable=True)
25 # Full name of the user visible file, relative to the repository root. 25 # Full name of the user visible file, relative to the repository root.
26 # Used by LFS. 26 # Used by LFS.
27 self.filename = path 27 self._revlog.filename = path
28 # Used by repo upgrade. 28 # Used by repo upgrade.
29 self.index = self._revlog.index 29 self.index = self._revlog.index
30 # Used by changegroup generation. 30 # Used by changegroup generation.
31 self._generaldelta = self._revlog._generaldelta 31 self._generaldelta = self._revlog._generaldelta
32 32
186 def verifyintegrity(self, state): 186 def verifyintegrity(self, state):
187 return self._revlog.verifyintegrity(state) 187 return self._revlog.verifyintegrity(state)
188 188
189 # TODO these aren't part of the interface and aren't internal methods. 189 # TODO these aren't part of the interface and aren't internal methods.
190 # Callers should be fixed to not use them. 190 # Callers should be fixed to not use them.
191
192 # Used by LFS.
193 @property
194 def filename(self):
195 return self._revlog.filename
196
197 @filename.setter
198 def filename(self, value):
199 self._revlog.filename = value
200 191
201 # Used by bundlefilelog, unionfilelog. 192 # Used by bundlefilelog, unionfilelog.
202 @property 193 @property
203 def indexfile(self): 194 def indexfile(self):
204 return self._revlog.indexfile 195 return self._revlog.indexfile