D2216: httppeer: remove httpspeer

2018-02-14 Thread indygreg (Gregory Szorc)
This revision was automatically updated to reflect the committed changes. Closed by commit rHGf6c7733655fc: httppeer: remove httpspeer (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D2216?vs=5615=5742 REVISION DETAIL

D2216: httppeer: remove httpspeer

2018-02-13 Thread indygreg (Gregory Szorc)
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

D2216: httppeer: remove httpspeer

2018-02-13 Thread lothiraldan (Boris Feld)
lothiraldan accepted this revision. lothiraldan added a comment. Could an extension import the `httpspeer` class or is it too low-level? REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D2216 To: indygreg, #hg-reviewers, lothiraldan Cc: lothiraldan,

D2216: httppeer: remove httpspeer

2018-02-12 Thread indygreg (Gregory Szorc)
indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY All it did was verify at construction time that Mercurial supports TLS. instance() is what's used to construct peer instances. So we can just inline this check