Update mutant exclude for lifetime syntax change
What the developer wrote
Authored by Jamil Lambert, PhD
Update mutant exclude for lifetime syntax change
Two mutant excludes stopped working and caused a timeout in the weekly
mutation testing. The lifetimes are now elided to fix a clippy error.
Change the excludes to match the new syntax and kill the mutants.
What changed, and why it matters
This commit only updates two text patterns in a mutation-testing configuration file. The patterns tell the mutation-testing tool which functions to skip. The function names changed slightly because of a Rust syntax cleanup (lifetime elision), so the skip patterns had to be updated to keep working. No program code, cryptography, network handling, or user-facing behavior was changed.
No security action needed. This is a test-configuration maintenance change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff modifies .cargo/mutants.toml, replacing two exclude_re entries that referenced WitnessEncoder<’a> with equivalent entries using the elided lifetime syntax WitnessEncoder<’_>. The comments explicitly state the reason: these excludes prevent cargo-mutants from introducing mutations that would cause infinite loops during testing. The change is purely a maintenance update to keep mutation testing exclusions aligned with source-code syntax changes; it does not alter any library code, API, or runtime logic.
Changed components
.cargo/mutants.tomlInspect captured patch +2 / −2
diff --git a/.cargo/mutants.toml b/.cargo/mutants.toml
index 26aa59d5..bd721b4b 100644
--- a/.cargo/mutants.toml
+++ b/.cargo/mutants.toml
@@ -46,6 +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.
+ "<impl Encoder for WitnessEncoder<'_>>::current_chunk", # Replacing the return with Some(vec![]) causes an infinite loop.
+ "<impl Encoder for WitnessEncoder<'_>>::advance", # Replacing the return with true causes an infinite loop.
]
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.