Stop using massive macros in toxav_basic_test.

[iphydf]

Turned a huge macro into a function. Macros are a pain to debug.

---

This change is 

[codecov]

Codecov Report

Merging #1120 into master will increase coverage by <.1%.
The diff coverage is 75%.

@@           Coverage Diff            @@
##           master   #1120     +/-   ##
========================================
+ Coverage    82.5%   82.5%   +<.1%     
========================================
  Files          80      80             
  Lines       14318   14339     +21     
========================================
+ Hits        11819   11838     +19     
- Misses       2499    2501      +2

Impacted Files	Coverage Δ
auto_tests/toxav_basic_test.c	83.5% <75%> (-1.4%)	⬇️
toxcore/group.c	71.7% <0%> (-1.5%)	⬇️
toxcore/Messenger.c	86.1% <0%> (-0.1%)	⬇️
toxcore/onion_client.c	95% <0%> (+0.6%)	⬆️
toxcore/LAN_discovery.c	84.9% <0%> (+0.9%)	⬆️
toxav/msi.c	65% <0%> (+1.1%)	⬆️
toxav/toxav.c	68.8% <0%> (+1.4%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update efcda6c...853a2a1. Read the comment docs.

[sudden6]

Reviewable status: 0 of 1 approvals obtained (waiting on @iphydf)

---

auto_tests/toxav_basic_test.c, line 107 at r1 (raw file):

    int a_br, int v_br)
{
    memset(AliceCC, 0, sizeof(CallControl));

memset on a struct is not very nice, at least add a todo/fixme comment

[iphydf]

Reviewable status: 1 change requests, 0 of 1 approvals obtained (waiting on @sudden6)

---

auto_tests/toxav_basic_test.c, line 107 at r1 (raw file):

Previously, sudden6 wrote…

memset on a struct is not very nice, at least add a todo/fixme comment

Done.

[sudden6]

Reviewed 1 of 1 files at r2.
Reviewable status: 1 change requests, 0 of 1 approvals obtained