diff options
| author | MarcoFalke <[email protected]> | 2020-05-09 07:30:27 -0400 |
|---|---|---|
| committer | MarcoFalke <[email protected]> | 2020-05-09 07:30:37 -0400 |
| commit | 88d8b4e182bfc75e8496f7046af7aab93307b9d0 (patch) | |
| tree | 51eae8b145f28db4efe1544d2d1be95df960e228 /src/script/script_error.cpp | |
| parent | Merge #18866: test: Fix verack race to avoid intermittent test failures (diff) | |
| parent | fuzz: use std::optional for sep_pos variable (diff) | |
| download | discoin-88d8b4e182bfc75e8496f7046af7aab93307b9d0.tar.xz discoin-88d8b4e182bfc75e8496f7046af7aab93307b9d0.zip | |
Merge #18901: fuzz: use std::optional for sep_pos_opt variable
420fa0770f37619bfa29898d59dac45b6a477abb fuzz: use std::optional for sep_pos variable (Harris)
Pull request description:
This PR changes the original `size_t sep_pos` to `std::optional<size_t> sep_post_opt` to remove the warning when compiling fuzz tests.
```shell
warning: variable 'sep_pos' may be uninitialized when used here [-Wconditional-uninitialized]
```
Also, it adds `--enable-c++17` flag to CI fuzz scripts.
ACKs for top commit:
practicalswift:
ACK 420fa0770f37619bfa29898d59dac45b6a477abb
MarcoFalke:
ACK 420fa07
Tree-SHA512: e967d5d8ab8ee7394b243ff5b28bac72d30bd14774e4a206f8c87474fad22769da76e4ba4e03cbef83b8f60e5293e9d9293b613e2e2e59e187d4e59ae6b874ca
Diffstat (limited to 'src/script/script_error.cpp')
0 files changed, 0 insertions, 0 deletions