Merge rust-bitcoin#5094: Add excludes for new mutants

8da0bec Add mutation exclude for timeouts (Jamil Lambert, PhD)
90ce314 Add mutation excludes for impl encoding::Decoder (Jamil Lambert, PhD)
d4b8d6c Make mutation testing exclude more specific (Jamil Lambert, PhD)

Pull request description:

  New mutants were found in the weekly mutation testing. One set is from a function that is for optimization and testing it is unnecessary, the other causes a timeout by changing the return of functions which are used in testing to exit loops.
  
  - Edit an existing exclude to make it more specific and only exclude the needed function.
  - Exclude the optimization functions `min_bytes_needed`.
  - Exclude the `WitnessEncoder` functions that cause an infinite loop in mutation testing.
  
  Closes rust-bitcoin#5088


ACKs for top commit:
  apoelstra:
    ACK 8da0bec; successfully ran local tests
  tcharding:
    ACK 8da0bec


Tree-SHA512: 8077157d29397a1b0f05bd29c1030086e0dfe3a0fca2d6252b1173cd9f99b49f425457638f1ec68931d4e24973a79b212bd150186d391aa3907d82debff04065

Author: apoelstra

.cargo/mutants.toml

@@ -11,7 +11,9 @@ exclude_re = [
     "deserialize", # Skip serde mutation tests
     "serde_details::<impl de::Visitor<'_>", # Skip serde mutation tests
     "Iterator", # Mutating operations in an iterator can result in an infinite loop
-    "impl encoding::Decodable", # Mutant replacing Default::default() is equivalent to returning new()
+    "<impl encoding::Decodable for .*>::decoder", # Mutant replacing Default::default() is equivalent to returning new()
+    "<impl encoding::Decoder for .*>::min_bytes_needed", # Function is just a cast for optimization.
+
 
     # ----------------------------------Crate-specific exclusions----------------------------------
     # Units
@@ -44,4 +46,6 @@ exclude_re = [
     "Script<T>::to_hex", # Deprecated
     "ScriptBuf::to_hex", # Deprecated
     "ScriptBuf<T>::to_hex", # Deprecated
+    "<impl Encoder for WitnessEncoder<'a>>::current_chunk", # Replacing the return with Some(vec![]) causes an infinite loop.
+    "<impl Encoder for WitnessEncoder<'a>>::advance", # Replacing the return with true causes an infinite loop.
 ]