forked from irungentoo/toxcore
-
Notifications
You must be signed in to change notification settings - Fork 292
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat: Add async event handling (callbacks) code.
Instead of synchronously handling events as they happen in `tox_iterate`, this first collects all events in a structure and then lets the client process them. This allows clients to process events in parallel, since the data structure returned is mostly immutable. This also makes toxcore compatible with languages that don't (easily) support callbacks from C into the non-C language. If we remove the callbacks, this allows us to add fields to the events without breaking the API.
- Loading branch information
Showing
25 changed files
with
536 additions
and
27 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,105 @@ | ||
| /* Auto Tests: Many clients. | ||
| */ | ||
|
|
||
| #include <stdio.h> | ||
| #include <stdlib.h> | ||
| #include <string.h> | ||
| #include <time.h> | ||
|
|
||
| #include "../testing/misc_tools.h" | ||
| #include "../toxcore/tox.h" | ||
| #include "../toxcore/tox_events.h" | ||
| #include "check_compat.h" | ||
|
|
||
| static bool await_message(Tox **toxes) | ||
| { | ||
| for (uint32_t i = 0; i < 100; ++i) { | ||
| // Ignore events on tox 1. | ||
| tox_events_free(tox_events_iterate(toxes[0])); | ||
| // Check if tox 2 got the message from tox 1. | ||
| Tox_Events *events = tox_events_iterate(toxes[1]); | ||
|
|
||
| if (events != nullptr) { | ||
| ck_assert(tox_events_get_friend_messages_size(events) == 1); | ||
| const Tox_Event_Friend_Message *msg_event = tox_events_get_friend_message(events, 0); | ||
| ck_assert(tox_event_friend_message_get_message_length(msg_event) == sizeof("hello")); | ||
| const uint8_t *msg = tox_event_friend_message_get_message(msg_event); | ||
| ck_assert_msg(memcmp(msg, "hello", sizeof("hello")) == 0, | ||
| "message was not expected 'hello' but '%s'", (const char *)msg); | ||
| tox_events_free(events); | ||
| return true; | ||
| } | ||
|
|
||
| c_sleep(tox_iteration_interval(toxes[0])); | ||
| } | ||
|
|
||
| return false; | ||
| } | ||
|
|
||
| static void test_tox_events(void) | ||
| { | ||
| uint8_t message[sizeof("hello")]; | ||
| memcpy(message, "hello", sizeof(message)); | ||
|
|
||
| Tox *toxes[2]; | ||
| uint32_t index[2]; | ||
|
|
||
| for (uint32_t i = 0; i < 2; ++i) { | ||
| index[i] = i + 1; | ||
| toxes[i] = tox_new_log(nullptr, nullptr, &index[i]); | ||
| tox_events_init(toxes[i]); | ||
| ck_assert_msg(toxes[i] != nullptr, "failed to create tox instances %u", i); | ||
| } | ||
|
|
||
| uint8_t pk[TOX_PUBLIC_KEY_SIZE]; | ||
| tox_self_get_dht_id(toxes[0], pk); | ||
| tox_bootstrap(toxes[1], "localhost", tox_self_get_udp_port(toxes[0], nullptr), pk, nullptr); | ||
|
|
||
| tox_self_get_public_key(toxes[0], pk); | ||
| tox_friend_add_norequest(toxes[1], pk, nullptr); | ||
|
|
||
| tox_self_get_public_key(toxes[1], pk); | ||
| tox_friend_add_norequest(toxes[0], pk, nullptr); | ||
|
|
||
| printf("bootstrapping and connecting 2 toxes\n"); | ||
|
|
||
| while (tox_self_get_connection_status(toxes[0]) == TOX_CONNECTION_NONE || | ||
| tox_self_get_connection_status(toxes[1]) == TOX_CONNECTION_NONE) { | ||
| // Ignore connection events for now. | ||
| tox_events_free(tox_events_iterate(toxes[0])); | ||
| tox_events_free(tox_events_iterate(toxes[1])); | ||
|
|
||
| c_sleep(tox_iteration_interval(toxes[0])); | ||
| } | ||
|
|
||
| printf("toxes online, waiting for friend connection\n"); | ||
|
|
||
| while (tox_friend_get_connection_status(toxes[0], 0, nullptr) == TOX_CONNECTION_NONE || | ||
| tox_friend_get_connection_status(toxes[1], 0, nullptr) == TOX_CONNECTION_NONE) { | ||
| // Ignore connection events for now. | ||
| tox_events_free(tox_events_iterate(toxes[0])); | ||
| tox_events_free(tox_events_iterate(toxes[1])); | ||
|
|
||
| c_sleep(tox_iteration_interval(toxes[0])); | ||
| } | ||
|
|
||
| printf("friends are connected via %s, now sending message\n", | ||
| tox_friend_get_connection_status(toxes[0], 0, nullptr) == TOX_CONNECTION_TCP ? "TCP" : "UDP"); | ||
|
|
||
| Tox_Err_Friend_Send_Message err; | ||
| tox_friend_send_message(toxes[0], 0, TOX_MESSAGE_TYPE_NORMAL, message, sizeof(message), &err); | ||
| ck_assert(err == TOX_ERR_FRIEND_SEND_MESSAGE_OK); | ||
|
|
||
| ck_assert(await_message(toxes)); | ||
|
|
||
| for (uint32_t i = 0; i < 2; ++i) { | ||
| tox_kill(toxes[i]); | ||
| } | ||
| } | ||
|
|
||
| int main(void) | ||
| { | ||
| setvbuf(stdout, nullptr, _IONBF, 0); | ||
| test_tox_events(); | ||
| return 0; | ||
| } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| #!/bin/sh | ||
|
|
||
| docker build -t toxchat/toxcore-autotools -f other/docker/autotools/Dockerfile . |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| #!/bin/sh | ||
|
|
||
| SANITIZER="${1:-asan}" | ||
|
|
||
| docker build -t toxchat/toxcore-circleci other/docker/circleci | ||
| docker run --rm -it -v "$PWD:/c-toxcore" toxchat/toxcore-circleci "$SANITIZER" |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,18 @@ | ||
| /* SPDX-License-Identifier: GPL-3.0-or-later | ||
| * Copyright © 2016-2021 The TokTok team. | ||
| */ | ||
| #include "ccompat.h" | ||
|
|
||
| #include <stdlib.h> | ||
| #include <string.h> | ||
|
|
||
| void *salloc(uint32_t size, const void *default_value) | ||
| { | ||
| void *ptr = malloc(size); | ||
|
|
||
| if (ptr != nullptr) { | ||
| memcpy(ptr, default_value, size); | ||
| } | ||
|
|
||
| return ptr; | ||
| } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.