diff options
| author | MarcoFalke <[email protected]> | 2020-02-16 09:22:01 -0800 |
|---|---|---|
| committer | MarcoFalke <[email protected]> | 2020-02-16 09:22:06 -0800 |
| commit | 33861a836727cc0514f9ec8de0e692b89f98c410 (patch) | |
| tree | 4b54d39e757252b582fb8e7331cf5ee97a701403 /src/httpserver.cpp | |
| parent | Merge #18108: Fix .gitignore policy in build_msvc directory (diff) | |
| parent | ci: use --enable-werror on more hosts (diff) | |
| download | discoin-33861a836727cc0514f9ec8de0e692b89f98c410.tar.xz discoin-33861a836727cc0514f9ec8de0e692b89f98c410.zip | |
Merge #18145: build: add Wreturn-type to Werror flags, check on more Travis machines
c98c26ee992f204b17bf17d271512b36c40ad8c5 ci: use --enable-werror on more hosts (Sjors Provoost)
6ba617dbe2b79d13ceec7d452e44ecab3bb814e7 build: add Wreturn-type to Werror flags (Sjors Provoost)
Pull request description:
I overlooked a missing `return false` in https://github.com/bitcoin/bitcoin/pull/17577#discussion_r379137776 and the warning only showed up on one Travis machine (`warning: control reaches end of non-void function [-Wreturn-type]`).
This PR promotes `Wreturn-type` to an error when configured with `--enable-werror`. I also added `--enable-werror` to the Travis machine that happened to catch this particular instance.
ACKs for top commit:
vasild:
ACK c98c26e.
practicalswift:
ACK c98c26ee992f204b17bf17d271512b36c40ad8c5
Tree-SHA512: 64e86c67fef2c5048aab201a8400b7e4a6f27b93d626159ba0b2807b5f119d2b0a83e3372db88f692cb4b0d059722d6a642d130c74a4f991a27f3a6b21780b5f
Diffstat (limited to 'src/httpserver.cpp')
0 files changed, 0 insertions, 0 deletions