1. Moved several macros from the header to the implementation file. They aren't...
authorscottmk <scottmk@9efc2488-bf62-4759-914b-345cdb29e865>
Sun, 25 Oct 2009 17:02:06 +0000 (17:02 +0000)
committerscottmk <scottmk@9efc2488-bf62-4759-914b-345cdb29e865>
Sun, 25 Oct 2009 17:02:06 +0000 (17:02 +0000)
commit9bec7fb6b42299d370f1af910193ae77b7ae6095
treea3549cde1e49c2bca8fa35c515cfbbca23fd9f2f
parentdebdcd7f724e971a30688b6f9e5a4d8cedc514cd
1. Moved several macros from the header to the implementation file.  They aren't used
anywhere else.

2. Renamed SERVER_SOCKET and CLIENT_SOCKET to LISTENER_SOCKET and DATA_SOCKET,
respectively.  The new names more accurately reflect the uses to which the two
socket types are put.  (Note that some so-called CLIENT_SOCKETs were, in fact,
opened by servers.)

3. Changed socket_open_udp_server() to open a DATA_SOCKET (formerly called a
CLIENT_SOCKET) instead of a LISTENER_SOCKET (formerly called a SERVER_SOCKET).
Otherwise an attempt to wait on such a socket would wind up treating it like
a listener.  That doesn't work for UDP.  In practice this change has no effect,
since no application ever calls this function anyway.

4. Always close a socket before removing the associated socket_node.  Otherwise we
will leak sockets in some situations.

5. Tinkered further with the comments, especially in the header file.

M    include/opensrf/socket_bundle.h
M    src/libopensrf/socket_bundle.c

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1825 9efc2488-bf62-4759-914b-345cdb29e865
include/opensrf/socket_bundle.h
src/libopensrf/socket_bundle.c