-
Notifications
You must be signed in to change notification settings - Fork 284
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move Networking_Core struct into the .c file. #662
Conversation
Without these, we'll have a memory leak on error paths.
These potentially cause uninitialised reads on some platforms or msgpack library versions.
We return E_WRITE because closing only fails when an I/O error occurs, which is likely an error from the write() call above. See close(2) for details. http://man7.org/linux/man-pages/man2/close.2.html
Also replace &(x) with &x for consistency.
This basically means: try until you run out of time (50 minutes on Travis). On Linux, we really want the tests to pass, so there is no point in limiting the number of retries. On windows, we don't retry, on FreeBSD it's limited to 1.
0f5d4dc
to
3e49fec
Compare
Reviewed 1 of 1 files at r1, 1 of 1 files at r2, 1 of 1 files at r3, 14 of 14 files at r4, 1 of 1 files at r5, 8 of 8 files at r6. toxcore/network.c, line 407 at r6 (raw file):
Wrong indentation. Should be 4 spaces. toxcore/network.c, line 411 at r6 (raw file):
Wrong indentation. Should be 4 spaces. toxcore/network.c, line 843 at r6 (raw file):
Wrong indentation. Should be 4 spaces. Comments from Reviewable |
3e49fec
to
d44feaf
Compare
Reviewed 1 of 1 files at r7. Comments from Reviewable |
To make it an abstract type everywhere except in network.c.
d44feaf
to
2fbed5b
Compare
To make it an abstract type everywhere except in network.c.
This change is