diff options
author | iphydf <iphydf@users.noreply.github.com> | 2020-03-14 03:49:28 +0000 |
---|---|---|
committer | iphydf <iphydf@users.noreply.github.com> | 2020-03-14 13:48:56 +0000 |
commit | 8795c5f987e2bd716d59a96418f1c7f6f9e83839 (patch) | |
tree | e3650c76847df0871e853964a9c54c16757b2205 /toxav | |
parent | 94d98ddd863279d5bf811d0107d73c373380dda2 (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.c | 2 | ||||
-rw-r--r-- | toxav/toxav_old.c | 12 |
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 @@ | |||
24 | int toxav_add_av_groupchat(Tox *tox, audio_data_cb *audio_callback, void *userdata) | 24 | int 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 | |||
88 | int toxav_groupchat_enable_av(Tox *tox, uint32_t groupnumber, audio_data_cb *audio_callback, void *userdata) | 94 | int 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 | |||
100 | int toxav_groupchat_disable_av(Tox *tox, uint32_t groupnumber) | 108 | int 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) | |||
109 | bool toxav_groupchat_av_enabled(Tox *tox, uint32_t groupnumber) | 119 | bool 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 | } |