aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorpracticalswift <practicalswift@users.noreply.github.com>2017-07-16 14:56:43 +0200
committerpracticalswift <practicalswift@users.noreply.github.com>2017-07-17 18:46:18 +0200
commitb82c55af78738258b56bd8fe7b5f8d5ccf85f832 (patch)
treed2d0e2d5c533bd9ff6a6ddf4ef5cce5ff8b9161c
parent5cfdda2503c995cdd563b1a2a29162ac298d173d (diff)
downloadbitcoin-b82c55af78738258b56bd8fe7b5f8d5ccf85f832.tar.xz
Add attribute [[noreturn]] (C++11) to functions that will not return
Rationale: * Reduce the number of false positives from static analyzers * Potentially enable additional compiler optimizations
-rw-r--r--src/random.cpp4
-rw-r--r--src/test/test_bitcoin_main.cpp8
2 files changed, 6 insertions, 6 deletions
diff --git a/src/random.cpp b/src/random.cpp
index 67efc7d945..b0044af514 100644
--- a/src/random.cpp
+++ b/src/random.cpp
@@ -39,10 +39,10 @@
#include <openssl/err.h>
#include <openssl/rand.h>
-static void RandFailure()
+[[noreturn]] static void RandFailure()
{
LogPrintf("Failed to read randomness, aborting\n");
- abort();
+ std::abort();
}
static inline int64_t GetPerformanceCounter()
diff --git a/src/test/test_bitcoin_main.cpp b/src/test/test_bitcoin_main.cpp
index 34beef5539..b556c953b9 100644
--- a/src/test/test_bitcoin_main.cpp
+++ b/src/test/test_bitcoin_main.cpp
@@ -10,14 +10,14 @@
std::unique_ptr<CConnman> g_connman;
-void Shutdown(void* parg)
+[[noreturn]] void Shutdown(void* parg)
{
- exit(EXIT_SUCCESS);
+ std::exit(EXIT_SUCCESS);
}
-void StartShutdown()
+[[noreturn]] void StartShutdown()
{
- exit(EXIT_SUCCESS);
+ std::exit(EXIT_SUCCESS);
}
bool ShutdownRequested()