diff options
| author | Garrett D'Amore <garrett@damore.org> | 2020-01-08 20:34:26 -0800 |
|---|---|---|
| committer | Garrett D'Amore <garrett@damore.org> | 2020-01-08 21:16:30 -0800 |
| commit | b21d7805523a407a14567017edbdef57ca81781f (patch) | |
| tree | e07f08bdc047ee4dfb057b670766e3de5bf2f981 /src/supplemental/http | |
| parent | 8479b4c8861c77cfd9eb64e724615605bdd1cbcb (diff) | |
| download | nng-b21d7805523a407a14567017edbdef57ca81781f.tar.gz nng-b21d7805523a407a14567017edbdef57ca81781f.tar.bz2 nng-b21d7805523a407a14567017edbdef57ca81781f.zip | |
fixes #1094 Consider in-lining task and aio
This only does it for rep, but it also has changes that should increase
the overall test coverage for the REP protocol
Diffstat (limited to 'src/supplemental/http')
| -rw-r--r-- | src/supplemental/http/http_client.c | 10 | ||||
| -rw-r--r-- | src/supplemental/http/http_conn.c | 8 | ||||
| -rw-r--r-- | src/supplemental/http/http_server.c | 20 |
3 files changed, 19 insertions, 19 deletions
diff --git a/src/supplemental/http/http_client.c b/src/supplemental/http/http_client.c index 3c60bd46..ecba84ae 100644 --- a/src/supplemental/http/http_client.c +++ b/src/supplemental/http/http_client.c @@ -91,7 +91,7 @@ http_dial_cb(void *arg) void nni_http_client_fini(nni_http_client *c) { - nni_aio_fini(c->aio); + nni_aio_free(c->aio); nng_stream_dialer_free(c->dialer); nni_mtx_fini(&c->mtx); NNI_FREE_STRUCT(c); @@ -132,7 +132,7 @@ nni_http_client_init(nni_http_client **cp, const nni_url *url) return (rv); } - if ((rv = nni_aio_init(&c->aio, http_dial_cb, c)) != 0) { + if ((rv = nni_aio_alloc(&c->aio, http_dial_cb, c)) != 0) { nni_http_client_fini(c); return (rv); } @@ -251,7 +251,7 @@ http_txn_reap(void *arg) } } nni_http_chunks_free(txn->chunks); - nni_aio_fini(txn->aio); + nni_aio_free(txn->aio); NNI_FREE_STRUCT(txn); } @@ -405,7 +405,7 @@ nni_http_transact_conn( nni_aio_finish_error(aio, NNG_ENOMEM); return; } - if ((rv = nni_aio_init(&txn->aio, http_txn_cb, txn)) != 0) { + if ((rv = nni_aio_alloc(&txn->aio, http_txn_cb, txn)) != 0) { NNI_FREE_STRUCT(txn); nni_aio_finish_error(aio, rv); return; @@ -450,7 +450,7 @@ nni_http_transact(nni_http_client *client, nni_http_req *req, nni_aio_finish_error(aio, NNG_ENOMEM); return; } - if ((rv = nni_aio_init(&txn->aio, http_txn_cb, txn)) != 0) { + if ((rv = nni_aio_alloc(&txn->aio, http_txn_cb, txn)) != 0) { NNI_FREE_STRUCT(txn); nni_aio_finish_error(aio, rv); return; diff --git a/src/supplemental/http/http_conn.c b/src/supplemental/http/http_conn.c index 1fc2c34e..583f5eee 100644 --- a/src/supplemental/http/http_conn.c +++ b/src/supplemental/http/http_conn.c @@ -690,8 +690,8 @@ nni_http_conn_fini(nni_http_conn *conn) } nni_mtx_unlock(&conn->mtx); - nni_aio_fini(conn->wr_aio); - nni_aio_fini(conn->rd_aio); + nni_aio_free(conn->wr_aio); + nni_aio_free(conn->rd_aio); nni_free(conn->rd_buf, conn->rd_bufsz); nni_mtx_fini(&conn->mtx); NNI_FREE_STRUCT(conn); @@ -716,8 +716,8 @@ http_init(nni_http_conn **connp, nng_stream *data) } conn->rd_bufsz = HTTP_BUFSIZE; - if (((rv = nni_aio_init(&conn->wr_aio, http_wr_cb, conn)) != 0) || - ((rv = nni_aio_init(&conn->rd_aio, http_rd_cb, conn)) != 0)) { + if (((rv = nni_aio_alloc(&conn->wr_aio, http_wr_cb, conn)) != 0) || + ((rv = nni_aio_alloc(&conn->rd_aio, http_rd_cb, conn)) != 0)) { nni_http_conn_fini(conn); return (rv); } diff --git a/src/supplemental/http/http_server.c b/src/supplemental/http/http_server.c index 06a93553..9a4ff2af 100644 --- a/src/supplemental/http/http_server.c +++ b/src/supplemental/http/http_server.c @@ -243,10 +243,10 @@ http_sconn_reap(void *arg) } nni_http_req_free(sc->req); nni_http_res_free(sc->res); - nni_aio_fini(sc->rxaio); - nni_aio_fini(sc->txaio); - nni_aio_fini(sc->txdataio); - nni_aio_fini(sc->cbaio); + nni_aio_free(sc->rxaio); + nni_aio_free(sc->txaio); + nni_aio_free(sc->txdataio); + nni_aio_free(sc->cbaio); // Now it is safe to release our reference on the server. nni_mtx_lock(&s->mtx); @@ -746,11 +746,11 @@ http_sconn_init(http_sconn **scp, nng_stream *stream) } if (((rv = nni_http_req_alloc(&sc->req, NULL)) != 0) || - ((rv = nni_aio_init(&sc->rxaio, http_sconn_rxdone, sc)) != 0) || - ((rv = nni_aio_init(&sc->txaio, http_sconn_txdone, sc)) != 0) || - ((rv = nni_aio_init(&sc->txdataio, http_sconn_txdatdone, sc)) != + ((rv = nni_aio_alloc(&sc->rxaio, http_sconn_rxdone, sc)) != 0) || + ((rv = nni_aio_alloc(&sc->txaio, http_sconn_txdone, sc)) != 0) || + ((rv = nni_aio_alloc(&sc->txdataio, http_sconn_txdatdone, sc)) != 0) || - ((rv = nni_aio_init(&sc->cbaio, http_sconn_cbdone, sc)) != 0)) { + ((rv = nni_aio_alloc(&sc->cbaio, http_sconn_cbdone, sc)) != 0)) { // Can't even accept the incoming request. Hard close. http_sconn_close(sc); return (rv); @@ -838,7 +838,7 @@ http_server_fini(nni_http_server *s) nni_mtx_unlock(&s->errors_mtx); nni_mtx_fini(&s->errors_mtx); - nni_aio_fini(s->accaio); + nni_aio_free(s->accaio); nni_mtx_fini(&s->mtx); nni_strfree(s->hostname); NNI_FREE_STRUCT(s); @@ -874,7 +874,7 @@ http_server_init(nni_http_server **serverp, const nni_url *url) nni_mtx_init(&s->errors_mtx); NNI_LIST_INIT(&s->errors, http_error, node); - if ((rv = nni_aio_init(&s->accaio, http_server_acccb, s)) != 0) { + if ((rv = nni_aio_alloc(&s->accaio, http_server_acccb, s)) != 0) { http_server_fini(s); return (rv); } |
