Fix and un-allow clippy::use_self lints (#235)

linebender · Dec 17, 2024 · 4154537 · 4154537
1 parent bef2115
commit 4154537
Show file tree

Hide file tree

Showing 6 changed files with 17 additions and 19 deletions.
diff --git a/fontique/src/generic.rs b/fontique/src/generic.rs
@@ -84,21 +84,21 @@ impl GenericFamily {
     }
 
     /// Returns a slice containing all generic family variants.
-    pub const fn all() -> &'static [GenericFamily] {
+    pub const fn all() -> &'static [Self] {
         &[
-            GenericFamily::SansSerif,
-            GenericFamily::Serif,
-            GenericFamily::Monospace,
-            GenericFamily::Cursive,
-            GenericFamily::Fantasy,
-            GenericFamily::SystemUi,
-            GenericFamily::UiSerif,
-            GenericFamily::UiSansSerif,
-            GenericFamily::UiMonospace,
-            GenericFamily::UiRounded,
-            GenericFamily::Emoji,
-            GenericFamily::Math,
-            GenericFamily::FangSong,
+            Self::SansSerif,
+            Self::Serif,
+            Self::Monospace,
+            Self::Cursive,
+            Self::Fantasy,
+            Self::SystemUi,
+            Self::UiSerif,
+            Self::UiSansSerif,
+            Self::UiMonospace,
+            Self::UiRounded,
+            Self::Emoji,
+            Self::Math,
+            Self::FangSong,
         ]
     }
 }

diff --git a/fontique/src/lib.rs b/fontique/src/lib.rs
@@ -21,7 +21,6 @@
 #![allow(clippy::exhaustive_enums)]
 #![allow(clippy::partial_pub_fields)]
 #![allow(clippy::shadow_unrelated)]
-#![allow(clippy::use_self)]
 
 #[cfg(not(any(feature = "std", feature = "libm")))]
 compile_error!("fontique requires either the `std` or `libm` feature to be enabled");

diff --git a/fontique/src/script.rs b/fontique/src/script.rs
@@ -17,7 +17,7 @@ pub struct Script(pub [u8; 4]);
 
 impl Script {
     /// Returns a mapping of scripts to sample text.
-    pub fn all_samples() -> &'static [(Script, &'static str)] {
+    pub fn all_samples() -> &'static [(Self, &'static str)] {
         SCRIPT_SAMPLES
     }
 

diff --git a/parley/src/layout/cursor.rs b/parley/src/layout/cursor.rs
@@ -479,7 +479,7 @@ impl Selection {
 
     /// Creates a new selection bounding the line at the given coordinates.
     pub fn line_from_point<B: Brush>(layout: &Layout<B>, x: f32, y: f32) -> Self {
-        let Selection { anchor, focus, .. } = Self::from_point(layout, x, y)
+        let Self { anchor, focus, .. } = Self::from_point(layout, x, y)
             .line_start(layout, false)
             .line_end(layout, true);
         Self {

diff --git a/parley/src/lib.rs b/parley/src/lib.rs
@@ -90,7 +90,6 @@
 #![allow(clippy::missing_assert_message)]
 #![allow(clippy::missing_panics_doc)]
 #![allow(clippy::shadow_unrelated)]
-#![allow(clippy::use_self)]
 
 #[cfg(not(any(feature = "std", feature = "libm")))]
 compile_error!("parley requires either the `std` or `libm` feature to be enabled");

diff --git a/parley/src/tests/utils/env.rs b/parley/src/tests/utils/env.rs
@@ -115,7 +115,7 @@ impl TestEnv {
         collection
             .family_id(DEFAULT_FONT_NAME)
             .unwrap_or_else(|| panic!("{} font not found", DEFAULT_FONT_NAME));
-        TestEnv {
+        Self {
             test_name: test_name.to_string(),
             check_counter: 0,
             font_cx: FontContext {