diff options
| author | practicalswift <[email protected]> | 2017-06-01 16:13:35 +0200 |
|---|---|---|
| committer | practicalswift <[email protected]> | 2017-09-22 14:59:27 +0200 |
| commit | 9ad6746ccd6dc31141fd0144686b641e31bf626b (patch) | |
| tree | a5f2a33bef9eae6fde623ba8fd9233f1bb9942a4 /src/validation.cpp | |
| parent | Merge #10888: range-based loops and const qualifications in net.cpp (diff) | |
| download | discoin-9ad6746ccd6dc31141fd0144686b641e31bf626b.tar.xz discoin-9ad6746ccd6dc31141fd0144686b641e31bf626b.zip | |
Use static_cast instead of C-style casts for non-fundamental types
A C-style cast is equivalent to try casting in the following order:
1. const_cast(...)
2. static_cast(...)
3. const_cast(static_cast(...))
4. reinterpret_cast(...)
5. const_cast(reinterpret_cast(...))
By using static_cast<T>(...) explicitly we avoid the possibility
of an unintentional and dangerous reinterpret_cast. Furthermore
static_cast<T>(...) allows for easier grepping of casts.
Diffstat (limited to 'src/validation.cpp')
| -rw-r--r-- | src/validation.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/validation.cpp b/src/validation.cpp index 85a6af858..c79804074 100644 --- a/src/validation.cpp +++ b/src/validation.cpp @@ -1549,9 +1549,9 @@ int32_t ComputeBlockVersion(const CBlockIndex* pindexPrev, const Consensus::Para int32_t nVersion = VERSIONBITS_TOP_BITS; for (int i = 0; i < (int)Consensus::MAX_VERSION_BITS_DEPLOYMENTS; i++) { - ThresholdState state = VersionBitsState(pindexPrev, params, (Consensus::DeploymentPos)i, versionbitscache); + ThresholdState state = VersionBitsState(pindexPrev, params, static_cast<Consensus::DeploymentPos>(i), versionbitscache); if (state == THRESHOLD_LOCKED_IN || state == THRESHOLD_STARTED) { - nVersion |= VersionBitsMask(params, (Consensus::DeploymentPos)i); + nVersion |= VersionBitsMask(params, static_cast<Consensus::DeploymentPos>(i)); } } |