Patchwork [1,of,3] sslutil: remove out of place comment

login
register
mail settings
Submitter Gregory Szorc
Date June 25, 2016, 4:37 p.m.
Message ID <3e53df5526513432e4c8.1466872633@ubuntu-vm-main>
Download mbox | patch
Permalink /patch/15612/
State Accepted
Headers show

Comments

Gregory Szorc - June 25, 2016, 4:37 p.m.
# HG changeset patch
# User Gregory Szorc <gregory.szorc@gmail.com>
# Date 1466865122 25200
#      Sat Jun 25 07:32:02 2016 -0700
# Node ID 3e53df5526513432e4c8eba52fb846062749c2bc
# Parent  591a8069b60e16f2331a53e9c608c37e14be6947
sslutil: remove out of place comment

This comment likely got orphaned as a result of refactoring in this
file. It isn't providing any useful value. So delete it.

Patch

diff --git a/mercurial/sslutil.py b/mercurial/sslutil.py
--- a/mercurial/sslutil.py
+++ b/mercurial/sslutil.py
@@ -323,20 +323,16 @@  def _verifycert(cert, hostname):
                 certname = value.lower().encode('ascii')
             except UnicodeEncodeError:
                 return _('IDN in certificate not supported')
             if matchdnsname(certname):
                 return None
             return _('certificate is for %s') % certname
     return _('no commonName or subjectAltName found in certificate')
 
-
-# CERT_REQUIRED means fetch the cert from the server all the time AND
-# validate it against the CA store provided in web.cacerts.
-
 def _plainapplepython():
     """return true if this seems to be a pure Apple Python that
     * is unfrozen and presumably has the whole mercurial module in the file
       system
     * presumably is an Apple Python that uses Apple OpenSSL which has patches
       for using system certificate store CAs in addition to the provided
       cacerts file
     """