diff --git a/jid/src/lib.rs b/jid/src/lib.rs
index 082819a7..0aca6558 100644
--- a/jid/src/lib.rs
+++ b/jid/src/lib.rs
@@ -101,10 +101,11 @@ fn domain_check(mut domain: &str) -> Result<Cow<'_, str>, Error> {
     }
 
     // Then if this is an IPv6 address.
-    if domain.starts_with('[') && domain.ends_with(']') {
-        if Ipv6Addr::from_str(&domain[1..domain.len() - 1]).is_ok() {
-            return Ok(Cow::Borrowed(domain));
-        }
+    if domain.starts_with('[')
+        && domain.ends_with(']')
+        && Ipv6Addr::from_str(&domain[1..domain.len() - 1]).is_ok()
+    {
+        return Ok(Cow::Borrowed(domain));
     }
 
     // idna can handle the root dot for us, but we still want to remove it for normalization
diff --git a/jid/src/parts.rs b/jid/src/parts.rs
index c64ea790..4ae513e2 100644
--- a/jid/src/parts.rs
+++ b/jid/src/parts.rs
@@ -59,6 +59,7 @@ macro_rules! def_part_types {
 
         impl $name {
             #[doc = def_part_parse_doc!($name, str, "Depending on whether the contents are changed by normalisation operations, this function either returns a copy or a reference to the original data.")]
+            #[allow(clippy::new_ret_no_self)]
             pub fn new(s: &str) -> Result<Cow<'_, $borrowed>, Error> {
                 let part = $check_fn(s)?;
                 match part {