diff options
| author | fanquake <fanquake@gmail.com> | 2020-12-02 20:22:42 +0800 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2020-12-02 20:52:19 +0800 |
| commit | 0a13d15c143906bb6fd5d30c25df03ce5e20418d (patch) | |
| tree | d59aa8258a48fefe8535b56a23219103324f8a94 /share | |
| parent | 3693fccc3acd25dad29879605bfa1fa5d55a7c3a (diff) | |
| parent | 1e62350ca20898189904a88dfef9ea11ddcd8626 (diff) | |
Merge #20530: lint, refactor: Update cppcheck linter to c++17 and improve explicit usage
1e62350ca20898189904a88dfef9ea11ddcd8626 refactor: Improve use of explicit keyword (Fabian Jahr)
c502a6dbfb854ca827a5a3925394f9e09d29b898 lint: Use c++17 std in cppcheck linter (Fabian Jahr)
Pull request description:
I found the `extended-lint-cppcheck` linter still uses `std=c++11` when reviewing #20471. The only difference in the output after this change is one line is missing:
```
src/script/descriptor.cpp:159:5: warning: Struct 'PubkeyProvider' has a constructor with 1 argument that is not explicit. [noExplicitConstructor]
```
After some digging, I am still not sure why this one is ignored with c++17 when 40 other`noExplicitConstructor` warnings were still appearing.
In the second commit, I fix these warnings, adding `explicit` where appropriate and adding fixes to ignore otherwise.
ACKs for top commit:
practicalswift:
cr ACK 1e62350ca20898189904a88dfef9ea11ddcd8626: patch looks correct!
MarcoFalke:
review ACK 1e62350ca20898189904a88dfef9ea11ddcd8626
Tree-SHA512: dff7b324429a57160e217cf38d9ddbb6e70c6cb3d3e3e0bd4013d88e07afc2292c3df94d0acf7122e9d486322821682ecf15c8f2724a78667764c05d47f89a12
Diffstat (limited to 'share')
0 files changed, 0 insertions, 0 deletions