summaryrefslogtreecommitdiff
path: root/toxav
diff options
context:
space:
mode:
authoriphydf <iphydf@users.noreply.github.com>2020-03-14 03:49:28 +0000
committeriphydf <iphydf@users.noreply.github.com>2020-03-14 13:48:56 +0000
commit8795c5f987e2bd716d59a96418f1c7f6f9e83839 (patch)
treee3650c76847df0871e853964a9c54c16757b2205 /toxav
parent94d98ddd863279d5bf811d0107d73c373380dda2 (diff)
Avoid casting back and forth between void-ptr.
In windows network code, we implement inet_pton and inet_ntop, which take void pointers. We can do slightly better because we already know the type when we call these functions, so we can avoid casting between void pointer and the addr struct types.
Diffstat (limited to 'toxav')
-rw-r--r--toxav/toxav.c2
-rw-r--r--toxav/toxav_old.c12
2 files changed, 14 insertions, 0 deletions
diff --git a/toxav/toxav.c b/toxav/toxav.c
index ba27fe27..597d7bea 100644
--- a/toxav/toxav.c
+++ b/toxav/toxav.c
@@ -131,7 +131,9 @@ ToxAV *toxav_new(Tox *tox, Toxav_Err_New *error)
131 131
132 // TODO(iphydf): Don't rely on toxcore internals. 132 // TODO(iphydf): Don't rely on toxcore internals.
133 Messenger *m; 133 Messenger *m;
134 //!TOKSTYLE-
134 m = *(Messenger **)tox; 135 m = *(Messenger **)tox;
136 //!TOKSTYLE+
135 137
136 if (m->msi_packet) { 138 if (m->msi_packet) {
137 rc = TOXAV_ERR_NEW_MULTIPLE; 139 rc = TOXAV_ERR_NEW_MULTIPLE;
diff --git a/toxav/toxav_old.c b/toxav/toxav_old.c
index e5cfbadc..9ccd1365 100644
--- a/toxav/toxav_old.c
+++ b/toxav/toxav_old.c
@@ -24,7 +24,9 @@
24int toxav_add_av_groupchat(Tox *tox, audio_data_cb *audio_callback, void *userdata) 24int toxav_add_av_groupchat(Tox *tox, audio_data_cb *audio_callback, void *userdata)
25{ 25{
26 // TODO(iphydf): Don't rely on toxcore internals. 26 // TODO(iphydf): Don't rely on toxcore internals.
27 //!TOKSTYLE-
27 Messenger *m = *(Messenger **)tox; 28 Messenger *m = *(Messenger **)tox;
29 //!TOKSTYLE+
28 return add_av_groupchat(m->log, tox, m->conferences_object, audio_callback, userdata); 30 return add_av_groupchat(m->log, tox, m->conferences_object, audio_callback, userdata);
29} 31}
30 32
@@ -42,7 +44,9 @@ int toxav_join_av_groupchat(Tox *tox, uint32_t friendnumber, const uint8_t *data
42 audio_data_cb *audio_callback, void *userdata) 44 audio_data_cb *audio_callback, void *userdata)
43{ 45{
44 // TODO(iphydf): Don't rely on toxcore internals. 46 // TODO(iphydf): Don't rely on toxcore internals.
47 //!TOKSTYLE-
45 Messenger *m = *(Messenger **)tox; 48 Messenger *m = *(Messenger **)tox;
49 //!TOKSTYLE+
46 return join_av_groupchat(m->log, tox, m->conferences_object, friendnumber, data, length, audio_callback, userdata); 50 return join_av_groupchat(m->log, tox, m->conferences_object, friendnumber, data, length, audio_callback, userdata);
47} 51}
48 52
@@ -63,7 +67,9 @@ int toxav_group_send_audio(Tox *tox, uint32_t groupnumber, const int16_t *pcm, u
63 uint32_t sample_rate) 67 uint32_t sample_rate)
64{ 68{
65 // TODO(iphydf): Don't rely on toxcore internals. 69 // TODO(iphydf): Don't rely on toxcore internals.
70 //!TOKSTYLE-
66 Messenger *m = *(Messenger **)tox; 71 Messenger *m = *(Messenger **)tox;
72 //!TOKSTYLE+
67 return group_send_audio(m->conferences_object, groupnumber, pcm, samples, channels, sample_rate); 73 return group_send_audio(m->conferences_object, groupnumber, pcm, samples, channels, sample_rate);
68} 74}
69 75
@@ -88,7 +94,9 @@ int toxav_group_send_audio(Tox *tox, uint32_t groupnumber, const int16_t *pcm, u
88int toxav_groupchat_enable_av(Tox *tox, uint32_t groupnumber, audio_data_cb *audio_callback, void *userdata) 94int toxav_groupchat_enable_av(Tox *tox, uint32_t groupnumber, audio_data_cb *audio_callback, void *userdata)
89{ 95{
90 // TODO(iphydf): Don't rely on toxcore internals. 96 // TODO(iphydf): Don't rely on toxcore internals.
97 //!TOKSTYLE-
91 Messenger *m = *(Messenger **)tox; 98 Messenger *m = *(Messenger **)tox;
99 //!TOKSTYLE+
92 return groupchat_enable_av(m->log, tox, m->conferences_object, groupnumber, audio_callback, userdata); 100 return groupchat_enable_av(m->log, tox, m->conferences_object, groupnumber, audio_callback, userdata);
93} 101}
94 102
@@ -100,7 +108,9 @@ int toxav_groupchat_enable_av(Tox *tox, uint32_t groupnumber, audio_data_cb *aud
100int toxav_groupchat_disable_av(Tox *tox, uint32_t groupnumber) 108int toxav_groupchat_disable_av(Tox *tox, uint32_t groupnumber)
101{ 109{
102 // TODO(iphydf): Don't rely on toxcore internals. 110 // TODO(iphydf): Don't rely on toxcore internals.
111 //!TOKSTYLE-
103 Messenger *m = *(Messenger **)tox; 112 Messenger *m = *(Messenger **)tox;
113 //!TOKSTYLE+
104 return groupchat_disable_av(m->conferences_object, groupnumber); 114 return groupchat_disable_av(m->conferences_object, groupnumber);
105} 115}
106 116
@@ -109,6 +119,8 @@ int toxav_groupchat_disable_av(Tox *tox, uint32_t groupnumber)
109bool toxav_groupchat_av_enabled(Tox *tox, uint32_t groupnumber) 119bool toxav_groupchat_av_enabled(Tox *tox, uint32_t groupnumber)
110{ 120{
111 // TODO(iphydf): Don't rely on toxcore internals. 121 // TODO(iphydf): Don't rely on toxcore internals.
122 //!TOKSTYLE-
112 Messenger *m = *(Messenger **)tox; 123 Messenger *m = *(Messenger **)tox;
124 //!TOKSTYLE+
113 return groupchat_av_enabled(m->conferences_object, groupnumber); 125 return groupchat_av_enabled(m->conferences_object, groupnumber);
114} 126}