Rearrange Ldap/LdapConn methods

Place the very niche is_closed() at the end so the more important ones come first.
inejge · Dec 20, 2022 · fbec42f · fbec42f
1 parent dc8add9
commit fbec42f
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 11 deletions.
diff --git a/src/ldap.rs b/src/ldap.rs
@@ -192,11 +192,6 @@ impl Ldap {
         Ok((result, exop, sasl_creds))
     }
 
-    /// Check whether the underlying connection has been closed.
-    pub fn is_closed(&mut self) -> bool {
-        self.tx.is_closed()
-    }
-
     /// Use the provided `SearchOptions` with the next Search operation, which can
     /// be invoked directly on the result of this method. If this method is used in
     /// combination with a non-Search operation, the provided options will be silently
@@ -722,9 +717,15 @@ impl Ldap {
             .map(|_| ())?)
     }
 
+    /// Check whether the underlying connection has been closed.
+    pub fn is_closed(&mut self) -> bool {
+        self.tx.is_closed()
+    }
+
     /// Return the TLS peer certificate in DER format.
+    ///
     /// The method returns Ok(None) if no certificate was found or
-    /// the connection does not use TLS.
+    /// the connection does not use or support TLS.
     pub async fn get_peer_certificate(&mut self) -> Result<Option<Vec<u8>>> {
         #[cfg(any(feature = "tls-native", feature = "tls-rustls"))]
         {

diff --git a/src/sync.rs b/src/sync.rs
@@ -87,11 +87,6 @@ impl LdapConn {
         self
     }
 
-    /// See [`Ldap::is_closed()`](struct.Ldap.html#method.is_closed).
-    pub fn is_closed(&mut self) -> bool {
-        self.ldap.tx.is_closed()
-    }
-
     /// See [`Ldap::simple_bind()`](struct.Ldap.html#method.simple_bind).
     pub fn simple_bind(&mut self, bind_dn: &str, bind_pw: &str) -> Result<LdapResult> {
         let rt = &mut self.rt;
@@ -258,6 +253,11 @@ impl LdapConn {
         rt.block_on(async move { ldap.abandon(msgid).await })
     }
 
+    /// See [`Ldap::is_closed()`](struct.Ldap.html#method.is_closed).
+    pub fn is_closed(&mut self) -> bool {
+        self.ldap.tx.is_closed()
+    }
+
     /// See [`Ldap::get_peer_certificate()`](struct.Ldap.html#method.get_peer_certificate).
     pub fn get_peer_certificate(&mut self) -> Result<Option<Vec<u8>>> {
         let rt = &mut self.rt;