Remove unused derives and dead functions in schemars code

Author: jonasbb

serde_with/src/schemars_0_8.rs

@@ -1059,20 +1059,13 @@ mod timespan {
     // #[non_exhaustive] is not actually necessary here but it should
     // help avoid warnings about semver breakage if this ever changes.
     #[non_exhaustive]
-    #[derive(Copy, Clone, Debug, Eq, PartialEq)]
     pub enum TimespanTargetType {
         String,
         F64,
         U64,
         I64,
     }
 
-    impl TimespanTargetType {
-        pub const fn is_signed(self) -> bool {
-            !matches!(self, Self::U64)
-        }
-    }
-
     /// Internal helper trait used to constrain which types we implement
     /// `JsonSchemaAs<T>` for.
     pub trait TimespanSchemaTarget<F> {
@@ -1194,7 +1187,7 @@ impl TimespanTargetType {
             ..Default::default()
         };
 
-        if self == Self::String {
+        if matches!(self, Self::String) {
             number.metadata().write_only = true;
         } else {
             string.metadata().write_only = true;

serde_with/src/schemars_0_9.rs

@@ -1061,20 +1061,13 @@ mod timespan {
     // #[non_exhaustive] is not actually necessary here but it should
     // help avoid warnings about semver breakage if this ever changes.
     #[non_exhaustive]
-    #[derive(Copy, Clone, Debug, Eq, PartialEq)]
     pub enum TimespanTargetType {
         String,
         F64,
         U64,
         I64,
     }
 
-    impl TimespanTargetType {
-        pub const fn is_signed(self) -> bool {
-            !matches!(self, Self::U64)
-        }
-    }
-
     /// Internal helper trait used to constrain which types we implement
     /// `JsonSchemaAs<T>` for.
     pub trait TimespanSchemaTarget<F> {
@@ -1192,7 +1185,7 @@ impl TimespanTargetType {
             }
         });
 
-        if self == Self::String {
+        if matches!(self, Self::String) {
             number
                 .ensure_object()
                 .insert("writeOnly".into(), true.into());

serde_with/src/schemars_1.rs

@@ -1068,20 +1068,13 @@ mod timespan {
     // #[non_exhaustive] is not actually necessary here but it should
     // help avoid warnings about semver breakage if this ever changes.
     #[non_exhaustive]
-    #[derive(Copy, Clone, Debug, Eq, PartialEq)]
     pub enum TimespanTargetType {
         String,
         F64,
         U64,
         I64,
     }
 
-    impl TimespanTargetType {
-        pub const fn is_signed(self) -> bool {
-            !matches!(self, Self::U64)
-        }
-    }
-
     /// Internal helper trait used to constrain which types we implement
     /// `JsonSchemaAs<T>` for.
     pub trait TimespanSchemaTarget<F> {
@@ -1199,7 +1192,7 @@ impl TimespanTargetType {
             }
         });
 
-        if self == Self::String {
+        if matches!(self, Self::String) {
             number
                 .ensure_object()
                 .insert("writeOnly".into(), true.into());