diff options
| author | fanquake <[email protected]> | 2020-11-17 13:57:23 +0800 |
|---|---|---|
| committer | fanquake <[email protected]> | 2020-11-17 13:57:40 +0800 |
| commit | 3457054c61d5b74800797e2c26a8126a21352382 (patch) | |
| tree | 176e9a3ee43f96713bd0a86ecaf44df201023dbe /src/script/script_error.cpp | |
| parent | Merge #20405: p2p: avoid calculating onion address checksum when version is n... (diff) | |
| parent | swapped "is" for "==" in literal comparison (diff) | |
| download | discoin-3457054c61d5b74800797e2c26a8126a21352382.tar.xz discoin-3457054c61d5b74800797e2c26a8126a21352382.zip | |
Merge #20346: script: modify security-check.py to use "==" instead of "is" for literal comparison
b6121edf70a8d50fd16ddbba0c3168e5e49bfc2e swapped "is" for "==" in literal comparison (Tyler Chambers)
Pull request description:
In Python 3.8+ literal comparisons using "is" instead of "==" produce a SyntaxWarning [source](https://docs.python.org/3.8/whatsnew/3.8.html#changes-in-python-behavior).
I checked the entire devtools directory, this seems to be the only occurrence.
This is a small fix, but removes the SyntaxWarning.
Fixes: #20338
ACKs for top commit:
hebasto:
re-ACK b6121edf70a8d50fd16ddbba0c3168e5e49bfc2e, only squashed since my [previous](https://github.com/bitcoin/bitcoin/pull/20346#pullrequestreview-525934568) review.
practicalswift:
re-ACK b6121edf70a8d50fd16ddbba0c3168e5e49bfc2e: patch still looks correct
theStack:
utACK b6121edf70a8d50fd16ddbba0c3168e5e49bfc2e
Tree-SHA512: 82a43495d6552fbaa3b02b58f0930b049d27aa937fe44b47714e3c059f844cc494de20674557371cbccf24fb8873ecb7376fb965ae326847eed2b855ed2d59c6
Diffstat (limited to 'src/script/script_error.cpp')
0 files changed, 0 insertions, 0 deletions