Reorder derives

Make all the derives uniform, more-or-less most important thing first
and group the eq/ord/hash ones.

src/raw.rs

@@ -73,7 +73,7 @@ impl Deserialize for Pair {
 ///
 /// We do not carry the `keylen` around, we just create the `VarInt` length when serializing and
 /// deserializing.
-#[derive(Debug, PartialEq, Hash, Eq, Clone, Ord, PartialOrd)]
+#[derive(Debug, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
 #[cfg_attr(feature = "serde", serde(crate = "actual_serde"))]
 pub struct Key {
@@ -136,7 +136,7 @@ pub type ProprietaryType = u8;
 
 /// Proprietary keys (i.e. keys starting with 0xFC byte) with their internal
 /// structure according to BIP 174.
-#[derive(Clone, PartialEq, Eq, PartialOrd, Ord, Hash, Debug)]
+#[derive(Debug, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
 #[cfg_attr(feature = "serde", serde(crate = "actual_serde"))]
 pub struct ProprietaryKey<Subtype = ProprietaryType>

src/v0/map/global.rs

@@ -24,7 +24,7 @@ use crate::version::Version;
 use crate::{consts, raw, serialize, V0};
 
 /// The global key-value map.
-#[derive(Clone, Debug, PartialEq, Eq, Hash)]
+#[derive(Debug, Clone, PartialEq, Eq, Hash)]
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
 #[cfg_attr(feature = "serde", serde(crate = "actual_serde"))]
 pub struct Global {

src/v0/map/input.rs

@@ -28,7 +28,7 @@ use crate::{consts, io, raw, serialize};
 
 /// A key-value map for an input of the corresponding index in the unsigned
 /// transaction.
-#[derive(Clone, Default, Debug, PartialEq, Eq, Hash)]
+#[derive(Debuf, Default, Clone, PartialEq, Eq, Hash)]
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
 #[cfg_attr(feature = "serde", serde(crate = "actual_serde"))]
 pub struct Input {

src/v0/map/output.rs

@@ -20,7 +20,7 @@ use crate::{consts, io, raw, serialize};
 
 /// A key-value map for an output of the corresponding index in the unsigned
 /// transaction.
-#[derive(Clone, Default, Debug, PartialEq, Eq, Hash)]
+#[derive(Debug, Default, Clone, PartialEq, Eq, Hash)]
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
 #[cfg_attr(feature = "serde", serde(crate = "actual_serde"))]
 pub struct Output {

src/v0/miniscript/error.rs

@@ -209,7 +209,7 @@ impl From<bitcoin::key::Error> for InputError {
 }
 
 /// Return error type for [`Psbt::update_input_with_descriptor`]
-#[derive(Debug, PartialEq, Eq, PartialOrd, Ord, Hash, Clone, Copy)]
+#[derive(Debug, Copy, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
 pub enum UtxoUpdateError {
     /// Index out of bounds
     IndexOutOfBounds(usize, usize),
@@ -258,7 +258,7 @@ impl std::error::Error for UtxoUpdateError {
 }
 
 /// Return error type for [`Psbt::update_output_with_descriptor`]
-#[derive(Debug, PartialEq, Eq, PartialOrd, Ord, Hash, Clone, Copy)]
+#[derive(Debug, Copy, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
 pub enum OutputUpdateError {
     /// Index out of bounds
     IndexOutOfBounds(usize, usize),

src/v0/miniscript/mod.rs

@@ -1118,7 +1118,7 @@ fn update_item_with_descriptor_helper<F: PsbtFields>(
 }
 
 /// Sighash message(signing data) for a given psbt transaction input.
-#[derive(Debug, PartialEq, Eq, PartialOrd, Ord, Hash, Clone, Copy)]
+#[derive(Debug, Copy, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
 pub enum PsbtSighashMsg {
     /// Taproot Signature hash
     TapSighash(sighash::TapSighash),

src/v0/mod.rs

@@ -592,7 +592,7 @@ mod display_from_str {
 }
 
 /// Data required to call [`GetKey`] to get the private key to sign an input.
-#[derive(Clone, Debug, PartialEq, Eq)]
+#[derive(Debug, Clone, PartialEq, Eq)]
 #[non_exhaustive]
 pub enum KeyRequest {
     /// Request a private key using the associated public key.
@@ -740,7 +740,7 @@ impl From<bip32::Error> for GetKeyError {
 }
 
 /// The various output types supported by the Bitcoin network.
-#[derive(Clone, Copy, Debug, PartialEq, Eq, PartialOrd, Ord, Hash)]
+#[derive(Debug, Copy, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
 #[non_exhaustive]
 pub enum OutputType {
     /// An output of type: pay-to-pubkey or pay-to-pubkey-hash.
@@ -772,7 +772,7 @@ impl OutputType {
 }
 
 /// Signing algorithms supported by the Bitcoin network.
-#[derive(Clone, Copy, Debug, PartialEq, Eq, PartialOrd, Ord, Hash)]
+#[derive(Debug, Copy, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
 pub enum SigningAlgorithm {
     /// The Elliptic Curve Digital Signature Algorithm (see [wikipedia]).
     ///

src/v2/map/global.rs

@@ -32,7 +32,7 @@ const OUTPUTS_MODIFIABLE: u8 = 0x01 << 1;
 const SIGHASH_SINGLE: u8 = 0x01 << 2;
 
 /// The global key-value map.
-#[derive(Clone, Debug, PartialEq, Eq, Hash)]
+#[derive(Debug, Clone, PartialEq, Eq, Hash)]
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
 #[cfg_attr(feature = "serde", serde(crate = "actual_serde"))]
 pub struct Global {

src/v2/map/input.rs

@@ -33,7 +33,7 @@ use crate::{io, raw, serialize, v0};
 
 /// A key-value map for an input of the corresponding index in the unsigned
 /// transaction.
-#[derive(Clone, Debug, PartialEq, Eq, Hash)]
+#[derive(Debug, Clone, PartialEq, Eq, Hash)]
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
 #[cfg_attr(feature = "serde", serde(crate = "actual_serde"))]
 pub struct Input {

src/v2/map/output.rs

@@ -21,7 +21,7 @@ use crate::{io, raw, serialize, v0};
 
 /// A key-value map for an output of the corresponding index in the unsigned
 /// transaction.
-#[derive(Clone, Default, Debug, PartialEq, Eq, Hash)]
+#[derive(Debug, Default, Clone, PartialEq, Eq, Hash)]
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
 #[cfg_attr(feature = "serde", serde(crate = "actual_serde"))]
 pub struct Output {

src/v2/miniscript/mod.rs

@@ -1106,7 +1106,7 @@ fn update_item_with_descriptor_helper<F: PsbtFields>(
 }
 
 /// Sighash message(signing data) for a given psbt transaction input.
-#[derive(Debug, PartialEq, Eq, PartialOrd, Ord, Hash, Clone, Copy)]
+#[derive(Debug, Copy, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
 pub enum PsbtSighashMsg {
     /// Taproot Signature hash
     TapSighash(sighash::TapSighash),

src/v2/mod.rs

@@ -938,7 +938,7 @@ impl Psbt {
 }
 
 /// Data required to call [`GetKey`] to get the private key to sign an input.
-#[derive(Clone, Debug, PartialEq, Eq)]
+#[derive(Debug, Clone, PartialEq, Eq)]
 #[non_exhaustive]
 pub enum KeyRequest {
     /// Request a private key using the associated public key.
@@ -1086,7 +1086,7 @@ impl From<bip32::Error> for GetKeyError {
 }
 
 /// The various output types supported by the Bitcoin network.
-#[derive(Clone, Copy, Debug, PartialEq, Eq, PartialOrd, Ord, Hash)]
+#[derive(Debug, Copy, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
 #[non_exhaustive]
 pub enum OutputType {
     /// An output of type: pay-to-pubkey or pay-to-pubkey-hash.
@@ -1118,7 +1118,7 @@ impl OutputType {
 }
 
 /// Signing algorithms supported by the Bitcoin network.
-#[derive(Clone, Copy, Debug, PartialEq, Eq, PartialOrd, Ord, Hash)]
+#[derive(Debug, Copy, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
 pub enum SigningAlgorithm {
     /// The Elliptic Curve Digital Signature Algorithm (see [wikipedia]).
     ///

src/version.rs

@@ -9,7 +9,7 @@ use crate::prelude::Vec;
 use crate::serialize::{self, Deserialize, Serialize};
 
 /// The PSBT version.
-#[derive(Copy, PartialEq, Eq, Clone, Debug, PartialOrd, Ord, Hash)]
+#[derive(Debug, Copy, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
 #[cfg_attr(feature = "serde", serde(crate = "actual_serde"))]
 pub struct Version(u32);