diff options
| author | MacroFake <falke.marco@gmail.com> | 2022-08-02 12:30:41 +0200 |
|---|---|---|
| committer | MacroFake <falke.marco@gmail.com> | 2022-08-02 12:30:49 +0200 |
| commit | 816ca01650f4cc66a61ac2f9b0f8b74cd9cd0cf8 (patch) | |
| tree | 0bd138342549f9eeba717090923fd0315b0d6c6f /src/test/hash_tests.cpp | |
| parent | ce3b75690d10426e97d087ca600140a19bf180b7 (diff) | |
| parent | fa7bef2e80c9c290b3e97114cfa7afdea5cbd53c (diff) | |
Merge bitcoin/bitcoin#25736: univalue: Remove unused and confusing set*() return value
fa7bef2e80c9c290b3e97114cfa7afdea5cbd53c univalue: Remove unused and confusing set*() return value (MacroFake)
Pull request description:
The value is:
* currently unused, and useless without `[[nodiscard]]`
* confusing, because it is always `true`, unless a num-string is set
Instead of adding `[[nodiscard]]`, throw when setting is not possible.
ACKs for top commit:
shaavan:
ACK fa7bef2e80c9c290b3e97114cfa7afdea5cbd53c
aureleoules:
ACK fa7bef2e80c9c290b3e97114cfa7afdea5cbd53c.
Tree-SHA512: 0d74f96f34cb93b66019ab75e12334c964630cc83434f22e58cc7a4fff2ee96a5767e42ab37f08acb67aeacba6811b09c75f1edc68d5e903ccfc59b1c82de891
Diffstat (limited to 'src/test/hash_tests.cpp')
0 files changed, 0 insertions, 0 deletions