Mercurial > public > mercurial-scm > hg

comparison mercurial/httppeer.py @ 36219:a463f375f021

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
httppeer: remove redundant code to fetch capabilities _fetchcaps() is called by httppeer.instance(), which is the only instantiator of httppeer. Since _fetchcaps() always sets self._caps and since 197d10e157ce removed the fallback for cases where the remote doesn't support capabilities, we can remove some dead code from httppeer.capabilities(). Differential Revision: https://phab.mercurial-scm.org/D2215
author Gregory Szorc <gregory.szorc@gmail.com>
date Mon, 12 Feb 2018 17:10:58 -0800
parents e4ccd7a69f77
children 874209855f5c
comparison
equal deleted inserted replaced
36218:e4ccd7a69f77 36219:a463f375f021
219 # End of _basepeer interface. 219 # End of _basepeer interface.
220 220
221 # Begin of _basewirepeer interface. 221 # Begin of _basewirepeer interface.
222 222
223 def capabilities(self): 223 def capabilities(self):
224 if self._caps is None: 224 # self._fetchcaps() should have been called as part of peer
225 try: 225 # handshake. So self._caps should always be set.
226 self._fetchcaps() 226 assert self._caps is not None
227 except error.RepoError:
228 self._caps = set()
229 self.ui.debug('capabilities: %s\n' %
230 (' '.join(self._caps or ['none'])))
231 return self._caps 227 return self._caps
232 228
233 # End of _basewirepeer interface. 229 # End of _basewirepeer interface.
234 230
235 # look up capabilities only when needed 231 # look up capabilities only when needed