Small cleanups on freebsd-connect-refused patch.

There should be no need to call be_socket_enable: that does an
event_add().  What we really want to do is event_active(), to make
sure that the writecb is executed.

Also, there was one "} if () {" that was missing an else.

I've noted that the return value for evutil_socket_connect() is
getting screwy, but since that isn't an exported function, we can fix
it whenever.

1 files changed, 5 insertions, 7 deletions

diff --git a/bufferevent_sock.c b/bufferevent_sock.c
index 22e76032..d731086f 100644
--- a/bufferevent_sock.c
+++ b/bufferevent_sock.c
@@ -401,19 +401,17 @@ bufferevent_socket_connect(struct bufferevent *bev,
 			result = 0;
 			goto done;
 		}
-	} if (r == 1) {
+	} else if (r == 1) {
 		/* The connect succeeded already. How very BSD of it. */
 		result = 0;
 		bufev_p->connecting = 1;
 		event_active(&bev->ev_write, EV_WRITE, 1);
 	} else {
 		/* The connect failed already.  How very BSD of it. */
-		if (! be_socket_enable(bev, EV_WRITE)) {
-			bufev_p->connection_refused = 1;
-			bufev_p->connecting = 1;
-			result = 0;
-			goto done;
-		}
+		bufev_p->connection_refused = 1;
+		bufev_p->connecting = 1;
+		result = 0;
+		event_active(&bev->ev_write, EV_WRITE, 1);
 	}
 
 	goto done;