diff options
author | iphydf <iphydf@users.noreply.github.com> | 2016-09-30 15:15:50 +0100 |
---|---|---|
committer | iphydf <iphydf@users.noreply.github.com> | 2016-09-30 19:06:44 +0100 |
commit | 1977d56caaff40ea9bbf6754b69bec9539a5a969 (patch) | |
tree | db9c6d6c59dc0f91900dafe85dea1ab90f0ca8d9 /auto_tests/toxav_basic_test.c | |
parent | 949ef785a4e7aa7868c9605b6bbed15c3f4beab9 (diff) |
Remove return after no-return situation (and other cleanups).
Cleanups:
- Fix header guards to not use reserved names.
- Avoid name shadowing.
- Removed an unused variable found by avoiding name shadowing.
Diffstat (limited to 'auto_tests/toxav_basic_test.c')
-rw-r--r-- | auto_tests/toxav_basic_test.c | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/auto_tests/toxav_basic_test.c b/auto_tests/toxav_basic_test.c index 1043c9be..860ad402 100644 --- a/auto_tests/toxav_basic_test.c +++ b/auto_tests/toxav_basic_test.c | |||
@@ -226,11 +226,11 @@ START_TEST(test_AV_flows) | |||
226 | memset(&AliceCC, 0, sizeof(CallControl)); \ | 226 | memset(&AliceCC, 0, sizeof(CallControl)); \ |
227 | memset(&BobCC, 0, sizeof(CallControl)); \ | 227 | memset(&BobCC, 0, sizeof(CallControl)); \ |
228 | \ | 228 | \ |
229 | TOXAV_ERR_CALL rc; \ | 229 | TOXAV_ERR_CALL call_err; \ |
230 | toxav_call(AliceAV, 0, A_BR, V_BR, &rc); \ | 230 | toxav_call(AliceAV, 0, A_BR, V_BR, &call_err); \ |
231 | \ | 231 | \ |
232 | if (rc != TOXAV_ERR_CALL_OK) { \ | 232 | if (call_err != TOXAV_ERR_CALL_OK) { \ |
233 | printf("toxav_call failed: %d\n", rc); \ | 233 | printf("toxav_call failed: %d\n", call_err); \ |
234 | ck_assert(0); \ | 234 | ck_assert(0); \ |
235 | } \ | 235 | } \ |
236 | \ | 236 | \ |
@@ -241,11 +241,11 @@ START_TEST(test_AV_flows) | |||
241 | while (BobCC.state != TOXAV_FRIEND_CALL_STATE_FINISHED) { \ | 241 | while (BobCC.state != TOXAV_FRIEND_CALL_STATE_FINISHED) { \ |
242 | \ | 242 | \ |
243 | if (BobCC.incoming) { \ | 243 | if (BobCC.incoming) { \ |
244 | TOXAV_ERR_ANSWER rc; \ | 244 | TOXAV_ERR_ANSWER answer_err; \ |
245 | toxav_answer(BobAV, 0, A_BR, V_BR, &rc); \ | 245 | toxav_answer(BobAV, 0, A_BR, V_BR, &answer_err); \ |
246 | \ | 246 | \ |
247 | if (rc != TOXAV_ERR_ANSWER_OK) { \ | 247 | if (answer_err != TOXAV_ERR_ANSWER_OK) { \ |
248 | printf("toxav_answer failed: %d\n", rc); \ | 248 | printf("toxav_answer failed: %d\n", answer_err); \ |
249 | ck_assert(0); \ | 249 | ck_assert(0); \ |
250 | } \ | 250 | } \ |
251 | BobCC.incoming = false; \ | 251 | BobCC.incoming = false; \ |
@@ -254,11 +254,11 @@ START_TEST(test_AV_flows) | |||
254 | \ | 254 | \ |
255 | if (time(NULL) - start_time >= 1) { \ | 255 | if (time(NULL) - start_time >= 1) { \ |
256 | \ | 256 | \ |
257 | TOXAV_ERR_CALL_CONTROL rc; \ | 257 | TOXAV_ERR_CALL_CONTROL cc_err; \ |
258 | toxav_call_control(AliceAV, 0, TOXAV_CALL_CONTROL_CANCEL, &rc); \ | 258 | toxav_call_control(AliceAV, 0, TOXAV_CALL_CONTROL_CANCEL, &cc_err); \ |
259 | \ | 259 | \ |
260 | if (rc != TOXAV_ERR_CALL_CONTROL_OK) { \ | 260 | if (cc_err != TOXAV_ERR_CALL_CONTROL_OK) { \ |
261 | printf("toxav_call_control failed: %d\n", rc); \ | 261 | printf("toxav_call_control failed: %d\n", cc_err); \ |
262 | ck_assert(0); \ | 262 | ck_assert(0); \ |
263 | } \ | 263 | } \ |
264 | } \ | 264 | } \ |