diff options
author | Andrei Karas <akaras@inbox.ru> | 2016-05-04 22:46:17 +0300 |
---|---|---|
committer | Andrei Karas <akaras@inbox.ru> | 2016-05-04 22:46:17 +0300 |
commit | b0227c5cbf0dada9fc46ade54686da369629de5e (patch) | |
tree | 133cc32e9fa8582d9e819c2c6ffb924c7f1e3a02 | |
parent | bf9c58df65f48416dd7ae02cae96228ab6bec6d4 (diff) | |
download | mv-b0227c5cbf0dada9fc46ade54686da369629de5e.tar.gz mv-b0227c5cbf0dada9fc46ade54686da369629de5e.tar.bz2 mv-b0227c5cbf0dada9fc46ade54686da369629de5e.tar.xz mv-b0227c5cbf0dada9fc46ade54686da369629de5e.zip |
Add macroses returnNullptr and returnNullptrV
-rw-r--r-- | src/utils/checkutils.h | 24 | ||||
-rw-r--r-- | src/utils/checkutils_unittest.cc | 27 |
2 files changed, 51 insertions, 0 deletions
diff --git a/src/utils/checkutils.h b/src/utils/checkutils.h index 7b76da8f1..52545e5cb 100644 --- a/src/utils/checkutils.h +++ b/src/utils/checkutils.h @@ -73,6 +73,22 @@ return ret; \ } +#define returnNullptrV(val) \ + if ((val) == nullptr) \ + { \ + reportStack(__FILE__, __LINE__, __func__, \ + "Detected null value", #val); \ + return; \ + } + +#define returnNullptr(ret, val) \ + if ((val) == nullptr) \ + { \ + reportStack(__FILE__, __LINE__, __func__, \ + "Detected null value", #val); \ + return ret; \ + } + void reportStack(const char *const file, const unsigned int line, const char *const func, @@ -102,5 +118,13 @@ void reportStack(const char *const file, if (val) \ return ret; +#define returnNullptrV(val) \ + if ((val) == nullptr) \ + return; \ + +#define returnNullptr(ret, val) \ + if ((val) == nullptr) \ + return ret; \ + #endif // ENABLE_ASSERTS #endif // UTILS_CHECKUTILS_H diff --git a/src/utils/checkutils_unittest.cc b/src/utils/checkutils_unittest.cc index 4cce96725..981e08f72 100644 --- a/src/utils/checkutils_unittest.cc +++ b/src/utils/checkutils_unittest.cc @@ -58,6 +58,19 @@ static int testReturnTrue(const bool val) return 1; } +static int testReturnNullptr(void *val) +{ + returnNullptr(0, val); + return 1; +} + +static void testReturnNullptrV(void *val) +{ + flag = false; + returnNullptrV(val); + flag = true; +} + TEST_CASE("CheckUtils") { logger = new Logger; @@ -114,5 +127,19 @@ TEST_CASE("CheckUtils") REQUIRE(testReturnTrue(true) == 0); } + SECTION("returnNullptr") + { + REQUIRE(testReturnNullptr(nullptr) == 0); + REQUIRE(testReturnNullptr((void*)1) == 1); + } + + SECTION("returnNullptrV") + { + testReturnNullptrV(nullptr); + REQUIRE(flag == false); + testReturnNullptrV((void*)1); + REQUIRE(flag == true); + } + delete2(logger); } |