diff options
| author | Wladimir J. van der Laan <[email protected]> | 2017-08-17 22:59:56 +0200 |
|---|---|---|
| committer | Wladimir J. van der Laan <[email protected]> | 2017-08-18 09:01:16 +0200 |
| commit | 4afb5aa9e173991548be3627c6df258368d95ff3 (patch) | |
| tree | 8d8523dd68167f9a6299dd92cc9dd5e0fd162b23 /src/bench/checkqueue.cpp | |
| parent | Merge #10901: Fix constness of ArgsManager methods (diff) | |
| parent | Declare single-argument (non-converting) constructors "explicit" (diff) | |
| download | discoin-4afb5aa9e173991548be3627c6df258368d95ff3.tar.xz discoin-4afb5aa9e173991548be3627c6df258368d95ff3.zip | |
Merge #10969: Declare single-argument (non-converting) constructors "explicit"
64fb0ac Declare single-argument (non-converting) constructors "explicit" (practicalswift)
Pull request description:
Declare single-argument (non-converting) constructors `explicit`.
In order to avoid unintended implicit conversions.
For a more thorough discussion, see ["C.46: By default, declare single-argument constructors explicit"](http://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#c46-by-default-declare-single-argument-constructors-explicit) in the C++ Core Guidelines (Stroustrup & Sutter).
Tree-SHA512: e0c6922e56b11fa402621a38656d8b1122d16dd8f160e78626385373cf184ac7f26cb4c1851eca47e9b0dbd5e924e39a85c3cbdcb627a05ee3a655ecf5f7a0f1
Diffstat (limited to 'src/bench/checkqueue.cpp')
| -rw-r--r-- | src/bench/checkqueue.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/bench/checkqueue.cpp b/src/bench/checkqueue.cpp index 88a2a570f..b7ae5c2d5 100644 --- a/src/bench/checkqueue.cpp +++ b/src/bench/checkqueue.cpp @@ -67,7 +67,7 @@ static void CCheckQueueSpeedPrevectorJob(benchmark::State& state) prevector<PREVECTOR_SIZE, uint8_t> p; PrevectorJob(){ } - PrevectorJob(FastRandomContext& insecure_rand){ + explicit PrevectorJob(FastRandomContext& insecure_rand){ p.resize(insecure_rand.randrange(PREVECTOR_SIZE*2)); } bool operator()() |