From c565171593c2c3683bb267e191eb7e1a45accd9f Mon Sep 17 00:00:00 2001
From: Luuk Hendriks <mail@luukhendriks.eu>
Date: Mon, 26 Feb 2024 16:47:46 +0100
Subject: [PATCH] Provide default fn impls in FromAttribute trait

---
 src/bgp/path_attributes.rs | 16 +++++++++++-----
 src/bgp/workshop/route.rs  | 34 ++++------------------------------
 2 files changed, 15 insertions(+), 35 deletions(-)

diff --git a/src/bgp/path_attributes.rs b/src/bgp/path_attributes.rs
index 40fd3a1d..386fe830 100644
--- a/src/bgp/path_attributes.rs
+++ b/src/bgp/path_attributes.rs
@@ -341,15 +341,21 @@ impl PaMap {
 
 
 pub trait FromAttribute {
-    fn from_attribute(value: PathAttribute) -> Option<Self>
+    fn from_attribute(_value: PathAttribute) -> Option<Self>
     where
-        Self: Sized;
-    fn attribute_type() -> Option<PathAttributeType>;
+        Self: Sized {
+            None
+    }
+
+    fn attribute_type() -> Option<PathAttributeType> {
+        None
+    }
+
 }
 
 impl From<AttributesMap> for PaMap {
-    fn from(value: AttributesMap) -> Self {
-        Self { attributes: value }
+    fn from(attributes: AttributesMap) -> Self {
+        Self { attributes }
     }
 }
 
diff --git a/src/bgp/workshop/route.rs b/src/bgp/workshop/route.rs
index 038b55c7..5dcb0ac3 100644
--- a/src/bgp/workshop/route.rs
+++ b/src/bgp/workshop/route.rs
@@ -13,7 +13,7 @@ use crate::bgp::{
     message::{nlri::Nlri, update_builder::StandardCommunitiesList},
     path_attributes::{
         ExtendedCommunitiesList, Ipv6ExtendedCommunitiesList,
-        LargeCommunitiesList, PathAttribute, PathAttributeType,
+        LargeCommunitiesList, PathAttribute, 
     },
 };
 
@@ -241,29 +241,11 @@ impl<N: Clone + Hash + Debug> WorkshopAttribute<N> for Vec<Community> {
     }
 }
 
-impl FromAttribute for Vec<Community> {
-    fn from_attribute(_value: PathAttribute) -> Option<Self> {
-        None
-    }
-
-    fn attribute_type() -> Option<PathAttributeType> {
-        None
-    }
-}
+impl FromAttribute for Vec<Community> { }
 
 //------------ NlriWorkshop --------------------------------------------------
 
-impl FromAttribute for crate::bgp::message::nlri::Nlri<Vec<u8>> {
-    fn from_attribute(_value: PathAttribute) -> Option<Self>
-    where
-        Self: Sized {
-        None
-    }
-
-    fn attribute_type() -> Option<PathAttributeType> {
-        None
-    }
-}
+impl FromAttribute for crate::bgp::message::nlri::Nlri<Vec<u8>> { }
 
 impl<N: Clone + Hash> WorkshopAttribute<N> for crate::bgp::message::nlri::Nlri<Vec<u8>> {
     fn retrieve(attrs: &PaMap) -> Option<Self>
@@ -286,15 +268,7 @@ impl<N: Clone + Hash> WorkshopAttribute<N> for crate::bgp::message::nlri::Nlri<V
 
 //------------ NextHopWorkshop -----------------------------------------------
 
-impl FromAttribute for crate::bgp::types::NextHop {
-    fn attribute_type() -> Option<PathAttributeType> {
-        None
-    }
-
-    fn from_attribute(_value: PathAttribute) -> Option<Self> {
-        None
-    }
-}
+impl FromAttribute for crate::bgp::types::NextHop { }
 
 impl<N: Clone + Hash> WorkshopAttribute<N> for crate::bgp::types::NextHop {
     fn retrieve(attrs: &PaMap) -> Option<Self> {