Fix comment formatting and grammar issues

Signed-off-by: luohewuyang <[email protected]>

Author: luohewuyang

src/lib.rs

@@ -47,10 +47,10 @@ use macos as platform;
 ///
 /// ## Environment Variables
 ///
-/// | Env. Var.      | Description                                                                           |
-/// |----------------|---------------------------------------------------------------------------------------|
-/// | SSL_CERT_FILE  | File containing an arbitrary number of certificates in PEM format.                    |
-/// | SSL_CERT_DIR   | Colon separated list of directories containing certificate files.                     |
+/// | Env. Var.     | Description                                                                           |
+/// |---------------|---------------------------------------------------------------------------------------|
+/// | SSL_CERT_FILE | File containing an arbitrary number of certificates in PEM format.                    |
+/// | SSL_CERT_DIR  | Colon separated list of directories containing certificate files.                     |
 ///
 /// If **either** (or **both**) are set, certificates are only loaded from
 /// the locations specified via environment variables and not the platform-

src/macos.rs

@@ -14,7 +14,7 @@ pub fn load_native_certs() -> CertificateResult {
     //
     // So we collect the certificates in this order; as a map of
     // their DER encoding to what we'll do with them.  We don't
-    // overwrite existing elements, which mean User settings
+    // overwrite existing elements, which means User settings
     // trump Admin trump System, as desired.
 
     let mut result = CertificateResult::default();
@@ -40,11 +40,11 @@ pub fn load_native_certs() -> CertificateResult {
             let der = cert.to_der();
 
             // If there are no specific trust settings, the default
-            // is to trust the certificate as a root cert.  Weird API but OK.
+            // is to trust the certificate as a root cert. Weird API but OK.
             // The docs say:
             //
-            // "Note that an empty Trust Settings array means "always trust this cert,
-            //  with a resulting kSecTrustSettingsResult of kSecTrustSettingsResultTrustRoot".
+            // "Note that an empty Trust Settings array means 'always trust this cert,
+            //  with a resulting kSecTrustSettingsResult of kSecTrustSettingsResultTrustRoot'."
             let trusted = match ts.tls_trust_settings_for_certificate(&cert) {
                 Ok(trusted) => trusted.unwrap_or(TrustSettingsForCertificate::TrustRoot),
                 Err(err) => {

tests/common/mod.rs

@@ -1,6 +1,6 @@
 use std::env;
 
-/// Cargo, at least sometimes, sets SSL_CERT_FILE and SSL_CERT_DIR internally,
+/// Cargo, at least sometimes, sets SSL_CERT_FILE and SSL_CERT_DIR internally, as
 /// it uses OpenSSL. So, always unset both at the beginning of a test even if
 /// the test doesn't use either.
 ///