[pmix server] Markup pmix_server_connection.c:
1. Highlight functions that should go somewhere else.
2. Remove code duplication from pmix_server_recv_connect_ack
3. Other changes that tries to make code look nicer.
[pmix server] Get rid of excessive send/receive states of pmix peers.
WARNING: Code tested only for correct compilation, not for correct
execution. This is intermediate commit.
[pmix server] Cleanup of pmix_server_sendrecv.c
1. Make output messages slightly more verbosive and have the same format
(need to continue with this).
2. send_bytes/read_bytes now don't modify the peer state. This should be
done in the caller (handler) so there would be no confusion.
3. Reorder functions so the basic ones (send_bytes/read_bytes) goes
first.
WARNING: This commit may be broken. Intermediate changes.
[pmix server] Cleanups in pmix_server_sendrecv.c [no semantic changes]:
1. Cleanup and markup code. Highlight things that need to be isolated
for portability.
2. Small changes in send/read_bytes (fix compilation errors).
3. Cleanup pmix_server_send_handler code: format output, small other
changes.
[pmix server] Completely get rid of peers status.
There is significant amount of changes but they are formatting mostly.
And few deletes of peer->state references.
WARNING: Commit is tested for compilation correctness only.