| Age | Commit message (Collapse) | Author |
|---|
|
|
|
|
|
Thanks to: https://github.com/TokTok/c-toxcore/pull/1216
|
|
No use making this a macro. LTO will inline this anyway, if we care about
performance. Generally, we avoid function-like macros that can be functions.
|
|
See https://github.com/TokTok/toktok-stack/pull/38.
|
|
A lot of bootstrap nodes are down :(.
|
|
This reverts commit f87f8719d02a9adbc9413256fe22958267cfefee.
|
|
This resets friend search rate in two circumstances where it should be
reset:
* When the friend gets a dhtpk packet to us;
* When a friend connection to the friend goes down.
|
|
This CI can run Bazel tests, because it supports IPv6. This is nice,
because now we can run IPv6 tests on every PR.
|
|
We ignore the errors for now.
|
|
This runs in the same build as asan, so "asan" now stands for both asan
and ubsan.
|
|
|
|
|
|
We were mistakenly not making a rejoin attempt on freezing connections
due to all closest connections going down. This fixes that, and tweaks
the test. I've still only done tens rather than hundreds of tests, but
I'm fairly confident that the conference test now consistently passes.
|
|
|
|
|
|
Also remove branch filter for appveyor. We only have a single branch in
every repository. Development happens in forks.
|
|
|
|
|
|
Using the full path including the repo name.
|
|
|
|
|
|
Possibly this is what's happening in #1149.
|
|
|
|
We should avoid recursion, as it makes reasoning about stack growth
harder. This tool shows (currently) 4 (non-tail) recursive functions, at
least 2 of which are easy to fix.
|
|
|
|
Because `()` means "some unknown number of parameters".
|
|
This is for modules like groups to hook into to have their own state
management in the `tox_savedata` format.
|
|
|
|
|
|
* add freezing and unfreezing of peers
* add rejoin packet
* revise handling of temporary invited connections
* rename "peer kill" packet to "peer leave" packet
* test rejoining in conference test
* use custom clock in conference test
|
|
This uses a trick to get read-write access to `this` from a `const`
member function, similar to C++ `mutable`, but uglier.
|
|
|
|
|
|
|
|
|
|
Don't know why codes with macro dosen't work.
As it's only a few expensive, just code it without macro for now.
\#if (MIN_LOGGER_LEVEL == LOG_TRACE) || (MIN_LOGGER_LEVEL == LOG_DEBUG)
fprintf(stderr, "[%s] %s:%d(%s) %s\n", strlevel, file, line, func, message);
\#endif
|
|
* Remove ability to disable logging.
* Remove unused `NCURSES_FOUND`.
* Remove checks for stdbool and existence of int types.
These mostly exist, and if they don't, we'll just fail to compile.
* Remove checks for various functions.
* Add check for `explicit_bzero` and `memset_s`.
|
|
The raw clock isn't subject to NTP adjustments. It may run less
accurately than the regular monotonic clock. I see no reason why raw
would be better for tox than the normal one.
This avoids one piece of OS-specific ifdef'd code.
|
|
Can't trivially get rid of recursion here, since it's a non-linear
recursive function.
|
|
opencv2 is deprecated and homebrew ships with opencv3 now.
|
|
|
|
Also adds a test (auto_reconnect_test) which fails without this change.
|
|
This inverts the truthiness of the return value. Previously, 0 meant
`true` and -1 meant `false`. Now, `true` (1) means `true` and `false` (0)
means `false`.
|
|
|
|
|
|
|
|
We'll make it non-required, but we want to know about these failures so
we are incentivised to fix them.
|
|
|
|
Changes:
* 100 columns maximum (not strict, can be a bit more sometimes).
* No space after cast.
|
