simplyfy enum defaults

using `#[default]` marker from rustc 1.62 with which the implementation can
be simply derived.

Author: jnqnfe

pulse-binding/src/channelmap.rs

@@ -52,13 +52,14 @@ pub const POSITION_MASK_ALL: PositionMask = 0xffffffffffffffffu64;
 /// Note, certain aliases, specifically `Left`, `Right`, `Center` and `Subwoofer`, available in the
 /// equivalent C enum are not provided here, since Rust does not allow aliases.
 #[repr(C)]
-#[derive(Debug, Copy, Clone, PartialEq, Eq)]
+#[derive(Debug, Default, Copy, Clone, PartialEq, Eq)]
 #[derive(FromPrimitive, ToPrimitive)]
 pub enum Position {
     /* NOTE: This enum’s variants and variant values **must** remain identical to the `sys` crate
        (C API) equivalent */
 
     /// Invalid.
+    #[default]
     Invalid = -1,
 
     /// Mono.
@@ -180,13 +181,6 @@ pub enum Position {
     TopRearCenter,
 }
 
-impl Default for Position {
-    #[inline(always)]
-    fn default() -> Self {
-        Position::Invalid
-    }
-}
-
 /// Check is equal to `sys` equivalent
 #[test]
 fn pos_compare_capi() {

pulse-binding/src/format.rs

@@ -32,7 +32,7 @@ pub use capi::pa_prop_type_t as PropType;
 /// Represents the type of encoding used in a stream or accepted by a sink.
 #[repr(C)]
 #[non_exhaustive]
-#[derive(Debug, Copy, Clone, PartialEq, Eq)]
+#[derive(Debug, Default, Copy, Clone, PartialEq, Eq)]
 #[derive(FromPrimitive, ToPrimitive)]
 #[allow(non_camel_case_types)]
 pub enum Encoding {
@@ -62,6 +62,7 @@ pub enum Encoding {
     DTSHD_IEC61937,
 
     /// Represents an invalid encoding.
+    #[default]
     Invalid = -1,
 }
 
@@ -100,13 +101,6 @@ impl From<capi::pa_encoding_t> for Encoding {
     }
 }
 
-impl Default for Encoding {
-    #[inline(always)]
-    fn default() -> Self {
-        Encoding::Invalid
-    }
-}
-
 /// Represents the format of data provided in a stream or processed by a sink.
 pub struct Info {
     /// The actual C object.

pulse-binding/src/sample.rs

@@ -74,7 +74,7 @@ use crate::time::MicroSeconds;
 /// should be preferred over direct use of the endian-specific variants, for improved flexibility
 /// and avoidance of mistakes.
 #[repr(C)]
-#[derive(Debug, Copy, Clone, PartialEq, Eq)]
+#[derive(Debug, Default, Copy, Clone, PartialEq, Eq)]
 #[derive(FromPrimitive, ToPrimitive)]
 #[allow(non_camel_case_types)]
 pub enum Format {
@@ -108,6 +108,7 @@ pub enum Format {
     S24_32be,
 
     /// An invalid value.
+    #[default]
     Invalid = -1,
 }
 
@@ -148,13 +149,6 @@ impl From<capi::pa_sample_format_t> for Format {
     }
 }
 
-impl Default for Format {
-    #[inline(always)]
-    fn default() -> Self {
-        Format::Invalid
-    }
-}
-
 /// Endian-independent format identifiers, for big-endian systems.
 #[cfg(target_endian = "big")]
 mod ei_formats {