Merge #502: Use allow attribute

b481cba Use allow attribute (Tobin C. Harding)

Pull request description:

  Done in an effort to clear all clippy warnings.

  Add two allow attributes to quieten clippy. This may not be the best solution but it at least flags the code for future improvement and allows us to enable clippy in CI (along with other clippy work).

ACKs for top commit:
  apoelstra:
    ACK b481cba

Tree-SHA512: bbadca63a974f3b218e558d7fd788546fc79f4e8e433659256bdf8b5184e4587ac3dccd4b983d10c08aa5e0e550f31bd4dc6a7a16f3b5e82fececdabc9526ce2

Author: apoelstra

src/descriptor/mod.rs

@@ -738,6 +738,7 @@ impl Descriptor<DescriptorPublicKey> {
     ///
     /// For multipath descriptors it will return as many descriptors as there is
     /// "parallel" paths. For regular descriptors it will just return itself.
+    #[allow(clippy::blocks_in_if_conditions)]
     pub fn into_single_descriptors(self) -> Result<Vec<Descriptor<DescriptorPublicKey>>, Error> {
         // All single-path descriptors contained in this descriptor.
         let mut descriptors = Vec::new();

src/miniscript/types/extra_props.rs

@@ -1055,6 +1055,7 @@ fn sat_minus_option_dissat(
 //
 // Args are of form: (<max_sat_size>, <count_dissat_size>)
 // max_[dis]sat_size of form: (<cost_of_witness>, <cost_of_sciptsig>)
+#[allow(clippy::type_complexity)]
 fn sat_minus_dissat_witness(
     a: &(Option<(usize, usize)>, Option<(usize, usize)>),
     b: &(Option<(usize, usize)>, Option<(usize, usize)>),