clippy

cofd/app/src/component/integrity.rs

@@ -148,7 +148,7 @@ impl<Message> Component<Message, iced::Renderer> for IntegrityComponent {
 					text(fl!("werewolf", "flesh-touchstone")).size(H3_SIZE),
 					column![text_input(
 						"",
-						character.touchstones.get(0).unwrap_or(&String::new()),
+						character.touchstones.first().unwrap_or(&String::new()),
 						|str| Event::TouchstoneChanged(0, str),
 					)
 					.padding(INPUT_PADDING)]

cofd/app/src/component/merits.rs

@@ -87,15 +87,13 @@ impl<Message> Component<Message, iced::Renderer> for MeritComponent<Message> {
 		let skills = &character.skills();
 
 		let vec: Vec<Translated<Merit>> = vec
-			.iter()
-			.cloned()
-			.filter(|e| {
+			.iter().filter(|&e| {
 				character
 					.merits
 					.iter()
 					.filter(|(merit, _)| *merit == *e)
 					.count() == 0 && e.is_available(&character, attributes, skills)
-			})
+			}).cloned()
 			.map(Into::into)
 			.collect();
 

cofd/app/src/component/skills.rs

@@ -109,7 +109,7 @@ impl<Message> SkillsComponent<Message> {
 
 			col1 = col1.push(
 				button(text(flt("skill", Some(skill.name())).unwrap()).style(
-					if specialties.len() > 0 {
+					if !specialties.is_empty() {
 						theme::Text::Color(Color::from_rgb(0.0, 0.7, 0.0))
 					} else {
 						theme::Text::Default

cofd/app/src/component/traits.rs

@@ -9,7 +9,6 @@ use cofd::{character::ArmorStruct, prelude::*};
 use crate::{
 	fl,
 	i18n::flt,
-	widget::dots::{Shape, SheetDots},
 	Element, INPUT_PADDING,
 };
 

cofd/app/src/main.rs

@@ -200,7 +200,7 @@ impl Application for PlayerCompanionApp {
 			Message::Msg => {}
 
 			Message::Save => match self.save() {
-				Ok(_) => {}
+				Ok(()) => {}
 				Err(err) => {
 					log::error!("{}", err);
 				}

cofd/app/src/store.rs

@@ -32,13 +32,13 @@ impl Store {
 					std::fs::create_dir_all(dir.unwrap()).ok()?;
 				}
 
-				let store = Some(Self {
-					dirs
-				});
+
 			}
 		}
 
-		store
+		Some(Self {
+					dirs
+				})
 	}
 
 	pub fn get<T: for<'a> Deserialize<'a>>(&self, name: &str) -> anyhow::Result<Option<T>> {

cofd/app/src/view/overview.rs

@@ -272,36 +272,36 @@ where
 				if val.is_empty() {
 					character.conditions.remove(i);
 				} else {
-					vec_changed(i, val, &mut character.conditions)
+					vec_changed(i, val, &mut character.conditions);
 				}
 			}
 			Event::AspirationChanged(i, val) => {
 				if val.is_empty() {
 					character.aspirations.remove(i);
 				} else {
-					vec_changed(i, val, &mut character.aspirations)
+					vec_changed(i, val, &mut character.aspirations);
 				}
 			}
 			Event::SplatThingChanged(i, val) => match &mut character.splat {
 				Splat::Changeling(.., data) => {
 					if val.is_empty() {
 						data.frailties.remove(i);
 					} else {
-						vec_changed(i, val, &mut data.frailties)
+						vec_changed(i, val, &mut data.frailties);
 					}
 				}
 				Splat::Vampire(.., data) => {
 					if val.is_empty() {
 						data.banes.remove(i);
 					} else {
-						vec_changed(i, val, &mut data.banes)
+						vec_changed(i, val, &mut data.banes);
 					}
 				}
 				Splat::Mage(.., data) => {
 					if val.is_empty() {
 						data.obsessions.remove(i);
 					} else {
-						vec_changed(i, val, &mut data.obsessions)
+						vec_changed(i, val, &mut data.obsessions);
 					}
 				}
 				_ => (),
@@ -571,9 +571,7 @@ where
 					.into()
 			} else {
 				let reg: Vec<Translated<Regalia>> = all_regalia
-					.iter()
-					.cloned()
-					.filter(|reg| reg != sg)
+					.iter().filter(|&reg| reg != sg).cloned()
 					.map(Into::into)
 					.collect();
 

cofd/app/src/widget/track.rs

@@ -85,7 +85,7 @@ where
 			}
 		}
 
-		if row.children().len() > 0 {
+		if !row.children().is_empty() {
 			col = col.push(row);
 		}