Another TCP_test upgrade

[hugbubby]

Mostly documentation + comments. Some cases where code was removed
in exchange for more compact/less sprawly for loops. Introduced a
function that removed like 30 lines of repeated code.

Edit: I do not understand git and I do not understand why I am pring old commits.

---

This change is 

[iphydf]

Review status: 0 of 1 LGTMs obtained

---

auto_tests/TCP_test.c, line 48 at r1 (raw file):

}

void do_TCP_server_delay(TCP_Server *tcp_s, int delay)

static void

---

auto_tests/TCP_test.c, line 479 at r1 (raw file):

    c_sleep(50);

    //The connection status should be unconfirmed here because we have finished

Space after //.

---

auto_tests/TCP_test.c, line 494 at r1 (raw file):

        ck_assert_msg(tcp_con_status(conn) == TCP_CLIENT_CONFIRMED, "Wrong connection status. Expected: %d, is: %d",
                      TCP_CLIENT_CONFIRMED,
                      tcp_con_status(conn));

This line can be combined with the previous one (remove one newline).

---

Comments from Reviewable

[hugbubby]

Review status: 0 of 1 LGTMs obtained

---

auto_tests/TCP_test.c, line 48 at r1 (raw file):

Previously, iphydf wrote…

static void

Done.

---

auto_tests/TCP_test.c, line 479 at r1 (raw file):

Previously, iphydf wrote…

Space after //.

Keep forgetting that. Habit I guess.

---

auto_tests/TCP_test.c, line 494 at r1 (raw file):

Previously, iphydf wrote…

This line can be combined with the previous one (remove one newline).

Done.

---

Comments from Reviewable

[iphydf]

---

Reviewed 1 of 1 files at r2.
Review status: complete! 1 of 1 LGTMs obtained

---

Comments from Reviewable