diff options
| author | practicalswift <[email protected]> | 2017-08-01 12:22:41 +0200 |
|---|---|---|
| committer | practicalswift <[email protected]> | 2017-08-16 16:33:25 +0200 |
| commit | 64fb0ac016c7fd01c60c39af60f6431bac57f9ee (patch) | |
| tree | 8d8523dd68167f9a6299dd92cc9dd5e0fd162b23 /src/sync.h | |
| parent | Merge #10901: Fix constness of ArgsManager methods (diff) | |
| download | discoin-64fb0ac016c7fd01c60c39af60f6431bac57f9ee.tar.xz discoin-64fb0ac016c7fd01c60c39af60f6431bac57f9ee.zip | |
Declare single-argument (non-converting) constructors "explicit"
In order to avoid unintended implicit conversions.
Diffstat (limited to 'src/sync.h')
| -rw-r--r-- | src/sync.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/sync.h b/src/sync.h index ddaf62b3b..0871c5fb4 100644 --- a/src/sync.h +++ b/src/sync.h @@ -196,7 +196,7 @@ private: int value; public: - CSemaphore(int init) : value(init) {} + explicit CSemaphore(int init) : value(init) {} void wait() { @@ -267,7 +267,7 @@ public: CSemaphoreGrant() : sem(nullptr), fHaveGrant(false) {} - CSemaphoreGrant(CSemaphore& sema, bool fTry = false) : sem(&sema), fHaveGrant(false) + explicit CSemaphoreGrant(CSemaphore& sema, bool fTry = false) : sem(&sema), fHaveGrant(false) { if (fTry) TryAcquire(); |