chore: fix clippy issues

Author: franzos

litehtml-sys/build.rs

@@ -1,5 +1,5 @@
 use std::env;
-use std::path::PathBuf;
+use std::path::{Path, PathBuf};
 
 fn main() {
     let manifest_dir = PathBuf::from(env::var("CARGO_MANIFEST_DIR").unwrap());
@@ -20,7 +20,7 @@ fn main() {
 }
 
 /// Build from vendored C++ source in vendor/litehtml
-fn build_vendored(manifest_dir: &PathBuf, out_dir: &PathBuf, csrc_dir: &PathBuf) {
+fn build_vendored(manifest_dir: &Path, out_dir: &Path, csrc_dir: &Path) {
     let vendor_dir = manifest_dir.join("vendor/litehtml");
     let gumbo_src = vendor_dir.join("src/gumbo");
     let gumbo_include = gumbo_src.join("include");
@@ -154,7 +154,7 @@ fn build_vendored(manifest_dir: &PathBuf, out_dir: &PathBuf, csrc_dir: &PathBuf)
 }
 
 /// Build against system-installed litehtml
-fn build_system(out_dir: &PathBuf, csrc_dir: &PathBuf) {
+fn build_system(out_dir: &Path, csrc_dir: &Path) {
     // Find system litehtml via LITEHTML_DIR or common paths
     let litehtml_dir = env::var("LITEHTML_DIR").ok().map(PathBuf::from);
 
@@ -238,12 +238,7 @@ fn find_library(name: &str) -> Option<PathBuf> {
     None
 }
 
-fn generate_bindings(
-    csrc_dir: &PathBuf,
-    out_dir: &PathBuf,
-    vendored: bool,
-    manifest_dir: &PathBuf,
-) {
+fn generate_bindings(csrc_dir: &Path, out_dir: &Path, vendored: bool, manifest_dir: &Path) {
     let mut builder = bindgen::Builder::default()
         .header(csrc_dir.join("litehtml_c.h").to_str().unwrap())
         .allowlist_function("lh_.*")

litehtml/src/lib.rs

@@ -136,7 +136,7 @@ impl Default for Border {
     }
 }
 
-#[derive(Debug, Clone, Copy, PartialEq)]
+#[derive(Debug, Clone, Copy, Default, PartialEq)]
 pub struct Borders {
     pub left: Border,
     pub top: Border,
@@ -145,18 +145,6 @@ pub struct Borders {
     pub radius: BorderRadiuses,
 }
 
-impl Default for Borders {
-    fn default() -> Self {
-        Self {
-            left: Border::default(),
-            top: Border::default(),
-            right: Border::default(),
-            bottom: Border::default(),
-            radius: BorderRadiuses::default(),
-        }
-    }
-}
-
 #[derive(Debug, Clone, Copy, Default, PartialEq)]
 pub struct MediaFeatures {
     pub media_type: MediaType,
@@ -593,19 +581,14 @@ impl TextDecorationStyle {
 }
 
 /// CSS `text-decoration-thickness` computed value.
-#[derive(Debug, Clone, Copy, PartialEq)]
+#[derive(Debug, Clone, Copy, Default, PartialEq)]
 pub enum DecorationThickness {
+    #[default]
     Auto,
     FromFont,
     Length(f32),
 }
 
-impl Default for DecorationThickness {
-    fn default() -> Self {
-        Self::Auto
-    }
-}
-
 // ---------------------------------------------------------------------------
 // Opaque pointer wrappers (borrowed, read-only)
 // ---------------------------------------------------------------------------
@@ -1343,7 +1326,7 @@ unsafe extern "C" fn cb_get_default_font_name(user_data: *mut c_void) -> *const
         let bridge = bridge_from_user_data(user_data);
         bridge.default_font_name.as_ptr()
     }))
-    .unwrap_or(b"serif\0".as_ptr().cast::<c_char>())
+    .unwrap_or(c"serif".as_ptr())
 }
 
 unsafe extern "C" fn cb_draw_list_marker(

litehtml/src/selection.rs

@@ -212,7 +212,7 @@ impl<'doc> Selection<'doc> {
             let needs_update = self
                 .order_cache
                 .as_ref()
-                .map_or(true, |c| c.a != start.element || c.b != end.element);
+                .is_none_or(|c| c.a != start.element || c.b != end.element);
             if needs_update {
                 let a_before_b = is_before(&start.element(), &end.element());
                 self.order_cache = Some(OrderCache {