We need to keep the GMainContext referenced so we can be sure it's
still alive when we remove the sources we've attached to it.

oups, we need to free the incoming checks only after we process them, not inside the loop processing them...

Alexis Ballier authored 15 years ago and Youness Alaoui committed 14 years ago

POSIX (http://www.opengroup.org/onlinepubs/009695399/basedefs/netinet/in.h.html) specifies that sockaddr_in is defined in netinet/in.h. Include the header where this struct is used.

Thanks to Olivier for spotting this.

After 5 hours of debugging... finally! remember to always remove the weak pointers when not needed anymore!

Use weak pointers to make sure that the recv callback does not destroy our object, and if it does, make sure not to use the agent or PseudoTcpSocket to avoid a crash

Fix a attach/detach recv bug where if we detach our recv callback from the nice agent during the callback of the tcp_readable, we would end up having the loop consume all data from PseudoTcpSocket without notifying anyone of it. This now only consumes data if the callback is available, and calls the readable callback when we attach a new recv callback if the pseudo tcp socket was still readable and all its data wasn't consumed yet

return FALSE on g_error timers because new gcc refuses to compile without a return value from these functions

Get rid of NiceSocket from the candidate, and make it into an obscure gpointer.. then cast it (only twice) inside the code. This allows us to get rid of all those socket layer .h files from installation which were not needed to be public

Add pseudotcp and pseudotcp-verbose flags to the NICE_DEBUG env var to enable debug messages from pseudotcp socket too. And update the docs

Disable strict aliasing.. since gcc 4.4.3, there are many warnings/errors caused by strict aliasing.. and it's hard to get the code clean without using unions everywhere especially when dealing with sockaddr_storage and sockaddr_in* structures. Since this is a socket library mainly, and sockaddr was designed for aliasing, it's best to just disable strict aliasing for the lib

Always start the conn check timer and the keepalive timer, even if initially there are no freezed check

oups, correctly free the tcp clock/data when the component is freed, not only when the stream is manually removed through the API

Adding 'Since' gtk-doc tags to specify since which version each function has become available (for non 0.0.1 ones)