Trivial style change: more explicit conditional check
What the developer wrote
Authored by kdmukai
Trivial style change: more explicit conditional check
What changed, and why it matters
This is a one-line style change that rewrites an empty-list check to be more explicit. It does not change what the program does, only how the code reads. There is no security relevance.
No action needed. The change is stylistic only.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit changes if verified_derivation_paths != []: to if len(verified_derivation_paths) > 0: in psbt_parser.py. Both expressions are functionally equivalent for the list type used here; the change is cosmetic and has no effect on behavior, attack surface, or data flow.
Changed components
src/seedsigner/models/psbt_parser.pyInspect captured patch +1 / −1
### src/seedsigner/models/psbt_parser.py
@@ -1128,7 +1128,7 @@ def _reject_if_seed_cannot_sign(self):
# cosigner, ours among them). One verified input path is enough for the psbt to
# be signable.
for verified_derivation_paths in self.verified_input_derivation_paths:
- if verified_derivation_paths != []:
+ if len(verified_derivation_paths) > 0:
return
# There's nothing for this seed to signWhy 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.