fixing warning

Author: LCrossman

microBioRust/src/record.rs

@@ -1,5 +1,6 @@
 ///Shared generic record types to reduce duplication between gbk and embl
 ///Minimal initial introduction: defines generic containers and builders that mirror the existing API where possible
+
 use std::collections::{HashMap, HashSet};
 
 #[derive(Clone, Debug, PartialEq, Eq, Hash)]
@@ -21,15 +22,9 @@ impl RangeValue {
 
 ///Traits to unify attribute enums across formats. Existing enums can implement Into these trait views if needed
 pub trait HasStartStopStrand {
-    fn start(&self) -> Option<RangeValue> {
-        None
-    }
-    fn stop(&self) -> Option<RangeValue> {
-        None
-    }
-    fn strand(&self) -> Option<i8> {
-        None
-    }
+    fn start(&self) -> Option<RangeValue> { None }
+    fn stop(&self) -> Option<RangeValue> { None }
+    fn strand(&self) -> Option<i8> { None }
 }
 
 ///Generic attribute builders
@@ -44,18 +39,12 @@ where
     K: Eq + std::hash::Hash,
     V: Eq + std::hash::Hash,
 {
-    pub fn set_name(&mut self, name: String) {
-        self.name = Some(name);
-    }
-    pub fn get_name(&self) -> Option<&String> {
-        self.name.as_ref()
-    }
+    pub fn set_name(&mut self, name: String) { self.name = Some(name); }
+    pub fn get_name(&self) -> Option<&String> { self.name.as_ref() }
     pub fn add(&mut self, key: K, value: V) {
-        self.attributes.entry(key).or_default().insert(value);
-    }
-    pub fn get(&self, key: &K) -> Option<&HashSet<V>> {
-        self.attributes.get(key)
+        self.attributes.entry(key).or_insert_with(HashSet::new).insert(value);
     }
+    pub fn get(&self, key: &K) -> Option<&HashSet<V>> { self.attributes.get(key) }
 }
 
 ///Generic record and records container
@@ -73,17 +62,14 @@ pub struct GenericRecord<S, F, Q> {
 }
 
 impl<S, F, Q> GenericRecord<S, F, Q> {
-    pub fn is_empty(&self) -> bool {
-        self.id.is_empty() && self.seq.is_empty()
-    }
+    pub fn is_empty(&self) -> bool { self.id.is_empty() && self.seq.is_empty() }
 }
 
+#[allow(dead_code)]
 pub struct GenericRecords<R> {
     inner: R,
 }
 
 impl<R> GenericRecords<R> {
-    pub fn new(reader: R) -> Self {
-        Self { inner: reader }
-    }
+    pub fn new(reader: R) -> Self { Self { inner: reader } }
 }