From 4eb78d39c5acd33e419a6d9203557d309e3d7873 Mon Sep 17 00:00:00 2001 From: "Daniel P. Berrange" Date: Mon, 12 Dec 2011 16:52:28 +0000 Subject: Merge reds_accept_connection into reds_accept Neither reds_accept_connection or reds_accept are very long, so the split is pointless & increases code size for no gain. Merge them together to reduce code size * server/reds.c: Merge reds_accept_connection into reds_accept Signed-off-by: Daniel P. Berrange --- server/reds.c | 26 ++++++++------------------ 1 file changed, 8 insertions(+), 18 deletions(-) (limited to 'server') diff --git a/server/reds.c b/server/reds.c index acd8495f..10ebf193 100644 --- a/server/reds.c +++ b/server/reds.c @@ -2736,22 +2736,6 @@ error: return NULL; } -static RedLinkInfo *reds_accept_connection(int listen_socket) -{ - RedLinkInfo *link; - RedsStream *stream; - - if (!(link = __reds_accept_connection(listen_socket))) { - return NULL; - } - - stream = link->stream; - stream->read = stream_read_cb; - stream->write = stream_write_cb; - stream->writev = stream_writev_cb; - - return link; -} static void reds_accept_ssl_connection(int fd, int event, void *data) { @@ -2814,12 +2798,18 @@ error: static void reds_accept(int fd, int event, void *data) { RedLinkInfo *link; + RedsStream *stream; - link = reds_accept_connection(reds->listen_socket); - if (link == NULL) { + if (!(link = __reds_accept_connection(reds->listen_socket))) { red_printf("accept failed"); return; } + + stream = link->stream; + stream->read = stream_read_cb; + stream->write = stream_write_cb; + stream->writev = stream_writev_cb; + reds_handle_new_link(link); } -- cgit