diff --git a/src/bgp/message/open.rs b/src/bgp/message/open.rs
index 6c993a4..43303c9 100644
--- a/src/bgp/message/open.rs
+++ b/src/bgp/message/open.rs
@@ -29,13 +29,13 @@ pub struct OpenMessage<Octets> {
 /// ## Convenience methods
 ///
 /// * [`my_asn()`][`OpenMessage::my_asn`]: returns the 32bit ASN if present,
-/// otherwise falls back to the conventional 16bit ASN (though represented as
-/// the 32bit [`inetnum::asn::Asn`][`Asn`]);
+///   otherwise falls back to the conventional 16bit ASN (though represented
+///   as the 32bit [`inetnum::asn::Asn`][`Asn`]);
 /// * [`multiprotocol_ids()`][`OpenMessage::multiprotocol_ids`]: returns an
-/// iterator over all the AFI/SAFI combinations listed as Capability in the
-/// Optional Parameters. If this yields an empty iterator, one can assume the
-/// default (IPv4/Unicast) can be used, but it is up to the user to handle as
-/// such.
+///   iterator over all the AFI/SAFI combinations listed as Capability in the
+///   Optional Parameters. If this yields an empty iterator, one can assume
+///   the default (IPv4/Unicast) can be used, but it is up to the user to
+///   handle as such.
 //
 //  0                   1                   2                   3
 //  0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
diff --git a/src/bgp/message/update.rs b/src/bgp/message/update.rs
index fadabfe..79188ba 100644
--- a/src/bgp/message/update.rs
+++ b/src/bgp/message/update.rs
@@ -94,12 +94,12 @@ impl<Octs: Octets> AsRef<[u8]> for UpdateMessage<Octs> {
 /// To accommodate for these hassles, the following methods are provided:
 ///
 /// * [`nlris()`][`UpdateMessage::nlris`] and
-/// [`withdrawals()`][`UpdateMessage::withdrawals`],
-/// providing iterators over announced and withdrawn prefixes ;
+///   [`withdrawals()`][`UpdateMessage::withdrawals`],
+///   providing iterators over announced and withdrawn prefixes ;
 /// * [`next_hop()`][`UpdateMessage::next_hop`], returning the [`NextHop`] ;
 /// * [`all_communities()`][`UpdateMessage::all_communities`], returning an
-/// optional `Vec` containing all conventional, Extended and Large
-/// Communities, wrapped in the [`Community`] enum.
+///   optional `Vec` containing all conventional, Extended and Large
+///   Communities, wrapped in the [`Community`] enum.
 ///
 /// For the mandatory path attributes, we have:
 ///
@@ -1154,12 +1154,12 @@ impl Default for PduParseInfo {
 /// and non AddPath enabled withdrawals, this struct holds booleans for all
 /// three sections where AddPath PathIds might or might not occur:
 ///
-///  *  the conventional IPv4 Unicast sections in the PDU, which are either
-///  both AddPath enabled or not, and,
+///  * the conventional IPv4 Unicast sections in the PDU, which are either
+///    both AddPath enabled or not, and,
 ///
 ///  * the MP_REACH and MP_UNREACH path attributes, which might carry NLRI for
-///  different address families and (thus) might differ in being AddPath
-///  enabled or not.
+///    different address families and (thus) might differ in being AddPath
+///    enabled or not.
 #[derive(Copy, Clone, Default, Debug, Eq, Hash, Ord, PartialEq, PartialOrd)]
 #[cfg_attr(feature = "arbitrary", derive(arbitrary::Arbitrary))]
 #[cfg_attr(feature = "serde", derive(serde::Serialize))]
diff --git a/src/bgp/path_selection.rs b/src/bgp/path_selection.rs
index b254a4b..6639179 100644
--- a/src/bgp/path_selection.rs
+++ b/src/bgp/path_selection.rs
@@ -18,7 +18,7 @@ use super::{
 /// Comparing two `OrdRoute`s, wrapping two routes with different sets of path
 /// attributes, might still yield 'equal'. Instead, consider comparing on the
 /// `PaMap` and/or `Tiebreakerinfo` directly (or [`fn inner`]).
-#[derive(Copy, Clone, Debug, Hash)]
+#[derive(Copy, Clone, Debug)]
 pub struct OrdRoute<'a, OS> {
     pa_map: &'a PaMap,
     tiebreakers: TiebreakerInfo,