Burn incense before clippy

Author: LLFourn

schnorr_fun/src/frost/mod.rs

@@ -208,8 +208,7 @@ impl<H: Hash32, NG: NonceGen> Frost<H, NG> {
     ///
     /// Parameters:
     ///
-    /// - `frost_key`: the joint public key we are signing under. This can be an `EvenY` or `Normal`
-    ///    It will return the same nonce regardless.
+    /// - `frost_key`: the joint public key we are signing under. This can be an `EvenY` or `Normal` It will return the same nonce regardless.
     /// - `secret`: you're secret key share for the `frost_key`
     /// - `session_id`: a string of bytes that is **unique for each signing attempt**.
     ///

schnorr_fun/src/musig.rs

@@ -338,11 +338,10 @@ where
     ///
     /// Parameters:
     ///
-    /// - `agg_key`: the joint public key we are signing under. This can be an `XOnly` or `Normal`.
-    ///    It will return the same nonce regardless.
+    /// - `agg_key`: the joint public key we are signing under. This can be an `XOnly` or `Normal`. It will return the same nonce regardless.
     /// - `secret`: you're secret key as part of `agg_key`. This **must be the secret key you are
-    ///    going to sign with**. It cannot be an "untweaked" version of the signing key. It must be
-    ///    exactly equal to the secret key you pass to [`sign`] (the MuSig specification requires this).
+    ///   going to sign with**. It cannot be an "untweaked" version of the signing key. It must be
+    ///   exactly equal to the secret key you pass to [`sign`] (the MuSig specification requires this).
     /// - `session_id`: a string of bytes that is **unique for each signing attempt**.
     ///
     /// The application should decide upon a unique `session_id` per call to this function. If the

secp256kfun/src/macros.rs

@@ -30,8 +30,8 @@ macro_rules! s {
 /// - `<point> + <point>` adds two points
 /// - `<point> - <point>` subtracts one point from another
 /// - `<scalar_iter> .* <point_iter>` does a [dot product](https://en.wikipedia.org/wiki/Dot_product)
-///    between a list of points and scalars. If one list is shorter than the other then the excess
-///    points or scalars will be multiplied by 0. See [`op::point_scalar_dot_product`].
+///   between a list of points and scalars. If one list is shorter than the other then the excess
+///   points or scalars will be multiplied by 0. See [`op::point_scalar_dot_product`].
 ///
 /// The terms of the expression can be any variable followed by simple method calls, attribute
 /// access etc. If your term involves more expressions (anything involving specifying types using

secp256kfun/src/marker/secrecy.rs

@@ -5,7 +5,7 @@
 ///
 /// - [`Secret`]: This value must be kept secret from parties I interact with.
 /// - [`Public`]: This value is known or it would not harm my security if this
-///               value is known to all parties I interact with.
+///   value is known to all parties I interact with.
 ///
 /// Note this consideration is only important if you do operations on the value
 /// during an interaction with a party. So if you would like to keep scalar `x`

secp256kfun/src/point.rs

@@ -590,7 +590,7 @@ impl<TR: PointType, SL, SR, ZR> SubAssign<Point<TR, SR, ZR>> for Point<NonNormal
 
 impl<S: Secrecy> core::iter::Sum for Point<NonNormal, S, Zero> {
     fn sum<I: Iterator<Item = Self>>(mut iter: I) -> Self {
-        let mut sum = iter.next().unwrap_or(Point::zero());
+        let mut sum = iter.next().unwrap_or_default();
         for point in iter {
             sum += point;
         }