diff options
| author | Wladimir J. van der Laan <laanwj@protonmail.com> | 2020-08-28 16:44:45 +0200 |
|---|---|---|
| committer | Wladimir J. van der Laan <laanwj@protonmail.com> | 2020-08-28 16:50:57 +0200 |
| commit | 9632b7edc74834c3249d08e94f44d27a5db1f435 (patch) | |
| tree | 894e6ff6c0a19dd271bf6eac3daaffe648b21af2 /src/wallet | |
| parent | 22acd36d5365932bedd9d6ef23fbd616105383c9 (diff) | |
| parent | d3e8adfada889a3c9fba930086eda609509aca07 (diff) | |
| download | bitcoin-9632b7edc74834c3249d08e94f44d27a5db1f435.tar.xz | |
Merge #19739: refactor: remove c-string interfaces for DecodeBase58{Check}
d3e8adfada889a3c9fba930086eda609509aca07 util: remove c-string interfaces for DecodeBase58{Check} (Sebastian Falbesoner)
Pull request description:
This micro-PR gets rid of base58 function interfaces that are redundant in terms of c-string / std::string variants; the c-string interface for `DecodeBase58Check` is completely unused outside the base58 module, while the c-string interface for `DecodeBase58` is only used in unit tests, where an implicit conversion to std::string is not problematic.
ACKs for top commit:
practicalswift:
ACK d3e8adfada889a3c9fba930086eda609509aca07 -- patch looks correct
laanwj:
Code review ACK d3e8adfada889a3c9fba930086eda609509aca07
Tree-SHA512: 006a4a1e23b11385f60820c188b8e6b1634a182ca36e29a6580f72150214c65a3fdb273ec439165f26ba88a42d2bf5bab1cf3666a9eaee222fb4e1c00aeba433
Diffstat (limited to 'src/wallet')
0 files changed, 0 insertions, 0 deletions