Remove use of let_chains

They won't be stabilized in 1.64 :(

Author: paolobarbolini

src/packet.rs

@@ -80,13 +80,19 @@ impl<'a, V: IpVersion> EchoReplyPacket<'a, V> {
     /// Parse an IP packet containing an ICMP echo reply packet
     pub(crate) fn from_reply(source: V, buf: Cow<'a, [u8]>) -> Option<Self> {
         if V::IS_V4 {
-            if let Some(ip_packet) = Ipv4Packet::new(&buf) && let Some(icmp_packet) = IcmpPacket::new(ip_packet.payload()) && icmp_packet.get_icmp_type() == IcmpTypes::EchoReply {
+            if let Some(ip_packet) = Ipv4Packet::new(&buf) {
+                if let Some(icmp_packet) = IcmpPacket::new(ip_packet.payload()) {
+                    if icmp_packet.get_icmp_type() == IcmpTypes::EchoReply {
+                        // SAFETY: we just checked that the packet is valid
+                        return Some(unsafe { Self::from_reply_unchecked(source, buf) });
+                    }
+                }
+            }
+        } else if let Some(icmp_packet) = Icmpv6Packet::new(&buf) {
+            if icmp_packet.get_icmpv6_type() == Icmpv6Types::EchoReply {
                 // SAFETY: we just checked that the packet is valid
                 return Some(unsafe { Self::from_reply_unchecked(source, buf) });
-             }
-        } else if let Some(icmp_packet) = Icmpv6Packet::new(&buf) && icmp_packet.get_icmpv6_type() == Icmpv6Types::EchoReply {
-            // SAFETY: we just checked that the packet is valid
-            return Some(unsafe { Self::from_reply_unchecked(source, buf) });
+            }
         }
 
         None