- 06 Mar, 2014 3 commits
-
-
Olivier Crête authored
-
Philip Withnall authored
net/if_arp.h doesn’t exist on OS X, and hence causes cross-compilation to fail. It doesn’t appear to be needed for any of the functions currently used in interfaces.c anyway.
-
Philip Withnall authored
-
- 04 Mar, 2014 1 commit
-
-
Philip Withnall authored
If we emit signals from component_io_cb() in the middle of a read, it’s possible that one of those signals will be picked up by the client to perform another read. A likely candidate (and what was triggering the re-entrancy here) would be component-state-changed. Since signals are emitted synchronously, the second read would being inside the first, and trigger the anti-re-entrancy assertion.
-
- 25 Feb, 2014 9 commits
-
-
Olivier Crête authored
-
Olivier Crête authored
-
Olivier Crête authored
We wan't to make sure not to have a race where a thread blocks in a blocking receive or send while nice_agent_remove_stream() is called
-
Olivier Crête authored
Also, keep a pointer to the other structure for tests to use
-
Olivier Crête authored
-
Olivier Crête authored
Otherwise, it may be stuck iterating the context forever if all of the other sources are gone.
-
Olivier Crête authored
Don't re-create it.
-
Olivier Crête authored
-
Olivier Crête authored
This way, there can be no annoying re-entrancy in our code.
-
- 24 Feb, 2014 9 commits
-
-
Olivier Crête authored
This should prevent us from re-adding re-entrancy in the future
-
Olivier Crête authored
This way, there can be no annoying re-entrancy in our code.
-
Olivier Crête authored
-
Olivier Crête authored
-
Philip Withnall authored
Expand documentation for nice_agent_gather_candidates() and nice_agent_set_remote_credentials().
-
Philip Withnall authored
-
Philip Withnall authored
Ensure the agent’s context doesn’t get iterated while in the middle of reading a message, as that will corrupt the component->recv_messages state.
-
Philip Withnall authored
-
Philip Withnall authored
Use it instead of memcmp() to avoid comparing padding bytes.
-
- 21 Feb, 2014 2 commits
-
-
Olivier Crête authored
-
Olivier Crête authored
-
- 20 Feb, 2014 2 commits
-
-
Olivier Crête authored
-
Olivier Crête authored
-
- 19 Feb, 2014 1 commit
-
-
Olivier Crête authored
-
- 13 Feb, 2014 1 commit
-
-
Olivier Crête authored
-
- 05 Feb, 2014 2 commits
-
-
Olivier Crête authored
Otherwise it ran the "yes" command
-
Olivier Crête authored
It tends to cause false warnings
-
- 31 Jan, 2014 10 commits
-
-
Olivier Crête authored
-
Olivier Crête authored
And we get close to 10% perf boost
-
Olivier Crête authored
Looping through the list of send segment was the only thing from libnice showing up in my profiler. Keeping a list of segment that were never sent has made it disappear.
-
Olivier Crête authored
-
Olivier Crête authored
-
Olivier Crête authored
-
Olivier Crête authored
This is very important for reliable mode. Also use it in the GOutputStream so as to not get into the case where there is still some space in the TCP buffer, but not enough for one message. Also warn against this problem.
-
Olivier Crête authored
Also made the GIOStream into a singleton, it always returns the same one. Also make it impossible to create a GIOStream for a non-existing stream/component
-
Olivier Crête authored
It was used correctly only half the time anyway
-
Olivier Crête authored
We never send multiple messages to separate targets in practice, so this will simplify the code
-