c++: establish a single place for opaque forward references

include/libwebsockets.h

@@ -38,6 +38,12 @@ extern "C" {
 
 #include "lws_config.h"
 
+/* place for one-shot opaque forward references */
+
+typedef struct lws_sequencer lws_seq_t; /* opaque */
+typedef struct lws_sorted_usec_list lws_sorted_usec_list_t; /* opaque */
+typedef struct lws_dsh lws_dsh_t;
+
 /*
  * CARE: everything using cmake defines needs to be below here
  */

include/libwebsockets/lws-client.h

@@ -49,8 +49,6 @@ enum lws_client_connect_ssl_connection_flags {
 		 * */
 };
 
-typedef struct lws_sequencer lws_seq_t;
-
 /** struct lws_client_connect_info - parameters to connect with when using
  *				    lws_client_connect_via_info() */
 

include/libwebsockets/lws-dsh.h

@@ -41,8 +41,6 @@
  * All management structures exist inside the allocated buffer.
  */
 
-typedef struct lws_dsh lws_dsh_t;
-
 /**
  * lws_dsh_create() - Allocate a DSH buffer
  *

include/libwebsockets/lws-sequencer.h

@@ -56,8 +56,6 @@ typedef enum lws_seq_cb_return {
 	LWSSEQ_RET_DESTROY
 } lws_seq_cb_return_t;
 
-typedef struct lws_sequencer lws_seq_t; /* opaque */
-
 /*
  * handler for this sequencer.  Return 0 if OK else nonzero to destroy the
  * sequencer.  LWSSEQ_DESTROYED will be called back to the handler so it can

include/libwebsockets/lws-timeout-timer.h

@@ -194,7 +194,6 @@ lws_timed_callback_vh_protocol_us(struct lws_vhost *vh,
 				  lws_usec_t us);
 
 
-typedef struct lws_sorted_usec_list lws_sorted_usec_list_t;
 typedef void (*sul_cb_t)(lws_sorted_usec_list_t *sul);
 
 typedef struct lws_sorted_usec_list {

lib/core-net/private.h

@@ -272,8 +272,6 @@ typedef struct lws_dsh_obj_head {
 	int				kind;
 } lws_dsh_obj_head_t;
 
-typedef struct lws_dsh lws_dsh_t;
-
 typedef struct lws_dsh_obj {
 	lws_dll2_t			list;	/* must be first */
 	lws_dsh_t			*dsh;	/* invalid when on free list */