D2216: httppeer: remove httpspeer

indygreg (Gregory Szorc) phabricator at mercurial-scm.org
Tue Feb 13 12:08:47 EST 2018


indygreg added a comment.


  In https://phab.mercurial-scm.org/D2216#36814, @lothiraldan wrote:
  
  > Could an extension import the `httpspeer` class or is it too low-level?
  
  
  In theory, sure. But these classes are instantiated via `httppeer.instance()`. If anything, this change makes things friendlier for extensions because there is one less class to wrap and the remaining class doesn't inherit from a class that may be wrapped.

REPOSITORY
  rHG Mercurial

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

To: indygreg, #hg-reviewers, lothiraldan
Cc: lothiraldan, mercurial-devel


More information about the Mercurial-devel mailing list