diff options
| author | Wladimir J. van der Laan <[email protected]> | 2017-10-09 16:40:36 +0200 |
|---|---|---|
| committer | Wladimir J. van der Laan <[email protected]> | 2017-10-09 16:41:06 +0200 |
| commit | da0478e6e5d93d13178d098209f4397730b94065 (patch) | |
| tree | 023ff3382c2cb40df86fe1d133fad6b9d1d4a3c3 /src | |
| parent | Merge #11448: [gui] reset addrProxy/addrSeparateProxyTor if colon char missing (diff) | |
| parent | Improve readability of DecodeBase58Check(...) (diff) | |
| download | discoin-da0478e6e5d93d13178d098209f4397730b94065.tar.xz discoin-da0478e6e5d93d13178d098209f4397730b94065.zip | |
Merge #10961: Improve readability of DecodeBase58Check(...)
c6a995e Improve readability of DecodeBase58Check(...) (practicalswift)
Pull request description:
Use the more readable form ...
```c++
&vchRet[vchRet.size() - 4]
```
... instead of ...
```c++
&v.end()[-n]
```
Has the added benefit of eliminating a spurious static analyzer warning about improper use of negative values.
Tree-SHA512: 5895310c189e9322082c28f34342ff9a6c238e2cae3f204521111c8a7981bc555af60b42de082c91608c1125dfc244a65c4faf929249a067a51435e2be74cb39
Diffstat (limited to 'src')
| -rw-r--r-- | src/base58.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/base58.cpp b/src/base58.cpp index c2cc5d979..9d5a2f496 100644 --- a/src/base58.cpp +++ b/src/base58.cpp @@ -139,7 +139,7 @@ bool DecodeBase58Check(const char* psz, std::vector<unsigned char>& vchRet) } // re-calculate the checksum, ensure it matches the included 4-byte checksum uint256 hash = Hash(vchRet.begin(), vchRet.end() - 4); - if (memcmp(&hash, &vchRet.end()[-4], 4) != 0) { + if (memcmp(&hash, &vchRet[vchRet.size() - 4], 4) != 0) { vchRet.clear(); return false; } |