http: remove unused nng_http_handler_get_data

The data is now passed directly to the handler function.
nanomsg · Jan 12, 2025 · c34abc2 · c34abc2
1 parent e5721f6
commit c34abc2
Show file tree

Hide file tree

Showing 7 changed files with 1 addition and 101 deletions.
diff --git a/docs/man/libnng.3.adoc b/docs/man/libnng.3.adoc
@@ -93,32 +93,6 @@ be the case if WebSocket support is configured as well.
 In this case, it is possible to access functionality to support the creation of
 HTTP (and HTTP/S if TLS support is present) servers and clients.
 
-==== Common HTTP Functions
-
-The following functions are used to work with HTTP requests, responses,
-and connections.
-
-|===
-|xref:nng_http_conn_read_req.3http.adoc[nng_http_conn_read_req()]|read HTTP request
-|xref:nng_http_conn_read_res.3http.adoc[nng_http_conn_read_res()]|read HTTP response
-|xref:nng_http_conn_write_req.3http.adoc[nng_http_conn_write_req()]|write HTTP request
-|xref:nng_http_conn_write_res.3http.adoc[nng_http_conn_write_res()]|write HTTP response
-|xref:nng_http_req_get_data.3http.adoc[nng_http_req_get_data()]|get HTTP request body
-|xref:nng_http_req_set_data.3http.adoc[nng_http_req_set_data()]|set HTTP request body
-|xref:nng_http_res_copy_data.3http.adoc[nng_http_res_copy_data()]|copy HTTP response body
-|xref:nng_http_res_get_data.3http.adoc[nng_http_res_get_data()]|get HTTP response body
-|xref:nng_http_res_get_header.3http.adoc[nng_http_res_get_header()]|return HTTP response header
-|xref:nng_http_res_set_data.3http.adoc[nng_http_res_set_data()]|set HTTP response body
-|===
-
-==== HTTP Client Functions
-
-These functions are intended for use with HTTP client applications.
-
-|===
-|xref:nng_http_conn_transact.3http.adoc[nng_http_conn_transact()]|perform one HTTP transaction on connection
-|===
-
 ==== HTTP Server Functions
 
 These functions are intended for use with HTTP server applications.
@@ -127,7 +101,6 @@ These functions are intended for use with HTTP server applications.
 |xref:nng_http_handler_alloc.3http.adoc[nng_http_handler_alloc()]|allocate HTTP server handler
 |xref:nng_http_handler_collect_body.3http.adoc[nng_http_handler_collect_body()]|set HTTP handler to collect request body
 |xref:nng_http_handler_free.3http.adoc[nng_http_handler_free()]|free HTTP server handler
-|xref:nng_http_handler_get_data.3http.adoc[nng_http_handler_get_data()]|return extra data for HTTP handler
 |xref:nng_http_handler_set_data.3http.adoc[nng_http_handler_set_data()]|set extra data for HTTP handler
 |xref:nng_http_handler_set_host.3http.adoc[nng_http_handler_set_host()]|set host for HTTP handler
 |xref:nng_http_handler_set_method.3http.adoc[nng_http_handler_set_method()]|set HTTP handler method

diff --git a/docs/man/nng_http_handler_get_data.3http.adoc b/docs/man/nng_http_handler_get_data.3http.adoc
diff --git a/docs/man/nng_http_handler_set_data.3http.adoc b/docs/man/nng_http_handler_set_data.3http.adoc
@@ -28,8 +28,6 @@ void nng_http_handler_set_data(nng_http_handler *handler, void *data,
 
 The `nng_http_handler_set_data()` function is used to set an additional
 _data_ for the _handler_.
-The stored _data_ can be retrieved later in the handler function using
-xref:nng_http_handler_get_data.3http.adoc[`nng_http_handler_get_data()`].
 
 Additionally, when the handler is deallocated, if _dtor_ is not `NULL`,
 then it will be called with _data_ as its argument.
@@ -40,6 +38,5 @@ this function is deallocate any resources associated with _data_.
 
 [.text-left]
 xref:nng_http_handler_alloc.3http.adoc[nng_http_handler_alloc(3http)],
-xref:nng_http_handler_get_data.3http.adoc[nng_http_server_get_data(3http)],
 xref:nng_http_server_add_handler.3http.adoc[nng_http_server_add_handler(3http)],
 xref:nng.7.adoc[nng(7)]
diff --git a/include/nng/http.h b/include/nng/http.h
@@ -301,9 +301,6 @@ NNG_DECL void nng_http_handler_set_tree(nng_http_handler *);
 NNG_DECL void nng_http_handler_set_data(
     nng_http_handler *, void *, void (*)(void *));
 
-// nng_http_handler_get_data returns the data that was previously stored.
-NNG_DECL void *nng_http_handler_get_data(nng_http_handler *);
-
 // nng_http_server is a handle to an HTTP server instance.  Servers
 // only serve a single port / address at this time.
 

diff --git a/src/supplemental/http/http_api.h b/src/supplemental/http/http_api.h
@@ -294,15 +294,11 @@ extern void nni_http_handler_set_host(nni_http_handler *, const char *);
 extern void nni_http_handler_set_method(nni_http_handler *, const char *);
 
 // nni_http_handler_set_data sets an opaque data element on the handler,
-// which will be available to the callback via nni_http_handler_get_data.
+// which will be available to the handler function as argument.
 // The callback is an optional destructor, and will be called with the
 // data as its argument, when the handler is being destroyed.
 extern void nni_http_handler_set_data(nni_http_handler *, void *, nni_cb);
 
-// nni_http_handler_get_data returns the data that was previously stored
-// by nni_http_handler_set_data.
-extern void *nni_http_handler_get_data(nni_http_handler *);
-
 // nni_http_handler_get_uri returns the URI set on the handler.
 extern const char *nni_http_handler_get_uri(nni_http_handler *);
 

diff --git a/src/supplemental/http/http_public.c b/src/supplemental/http/http_public.c
@@ -434,17 +434,6 @@ nng_http_handler_set_data(nng_http_handler *h, void *dat, void (*dtor)(void *))
 #endif
 }
 
-void *
-nng_http_handler_get_data(nng_http_handler *h)
-{
-#ifdef NNG_SUPP_HTTP
-	return (nni_http_handler_get_data(h));
-#else
-	NNI_ARG_UNUSED(h);
-	return (NULL);
-#endif
-}
-
 int
 nng_http_server_hold(nng_http_server **srvp, const nng_url *url)
 {

diff --git a/src/supplemental/http/http_server.c b/src/supplemental/http/http_server.c
@@ -163,12 +163,6 @@ nni_http_handler_set_data(nni_http_handler *h, void *data, nni_cb dtor)
 	h->dtor = dtor;
 }
 
-void *
-nni_http_handler_get_data(nni_http_handler *h)
-{
-	return (h->data);
-}
-
 const char *
 nni_http_handler_get_uri(nni_http_handler *h)
 {