diff mbox series

[2/2] helper: add stubs for missing documentation

Message ID 1486121040-12393-2-git-send-email-petri.savolainen@linaro.org
State New
Headers show
Series [1/2] helper: linux: renamed threads_extn to linux helpers | expand

Commit Message

Petri Savolainen Feb. 3, 2017, 11:24 a.m. UTC
This suppress excessive amount of doxygen warnings caused by
missing helper documentation. Other more improtant warnings
may go unnoticed if these warning fill the screen.

Signed-off-by: Petri Savolainen <petri.savolainen@linaro.org>

---
 helper/include/odp/helper/odph_cuckootable.h   | 52 +++++++++++++++++++++++
 helper/include/odp/helper/odph_hashtable.h     | 57 ++++++++++++++++++++++++++
 helper/include/odp/helper/odph_iplookuptable.h | 57 +++++++++++++++++++++++++-
 helper/include/odp/helper/odph_lineartable.h   | 48 ++++++++++++++++++++++
 4 files changed, 212 insertions(+), 2 deletions(-)

-- 
2.8.1

Comments

Mike Holmes Feb. 3, 2017, 12:25 p.m. UTC | #1
Is this a duplicate of http://patches.opendataplane.org/patch/8030/

On 3 February 2017 at 06:24, Petri Savolainen
<petri.savolainen@linaro.org> wrote:
> This suppress excessive amount of doxygen warnings caused by

> missing helper documentation. Other more improtant warnings

> may go unnoticed if these warning fill the screen.

>

> Signed-off-by: Petri Savolainen <petri.savolainen@linaro.org>

> ---

>  helper/include/odp/helper/odph_cuckootable.h   | 52 +++++++++++++++++++++++

>  helper/include/odp/helper/odph_hashtable.h     | 57 ++++++++++++++++++++++++++

>  helper/include/odp/helper/odph_iplookuptable.h | 57 +++++++++++++++++++++++++-

>  helper/include/odp/helper/odph_lineartable.h   | 48 ++++++++++++++++++++++

>  4 files changed, 212 insertions(+), 2 deletions(-)

>

> diff --git a/helper/include/odp/helper/odph_cuckootable.h b/helper/include/odp/helper/odph_cuckootable.h

> index d569980..65cd555 100644

> --- a/helper/include/odp/helper/odph_cuckootable.h

> +++ b/helper/include/odp/helper/odph_cuckootable.h

> @@ -52,27 +52,79 @@

>  extern "C" {

>  #endif

>

> +/**

> + * documentation missing

> + *

> + * @param name

> + * @param capacity

> + * @param key_size

> + * @param value_size

> + *

> + * @return

> + */

>  odph_table_t odph_cuckoo_table_create(

>                 const char *name,

>                 uint32_t capacity,

>                 uint32_t key_size,

>                 uint32_t value_size);

>

> +/**

> + * documentation missing

> + *

> + * @param name

> + *

> + * @return

> + */

>  odph_table_t odph_cuckoo_table_lookup(const char *name);

>

> +/**

> + * documentation missing

> + *

> + * @param table

> + *

> + * @return

> + */

>  int odph_cuckoo_table_destroy(odph_table_t table);

>

> +/**

> + * documentation missing

> + *

> + * @param table

> + * @param key

> + * @param value

> + *

> + * @return

> + */

>  int odph_cuckoo_table_put_value(

>                 odph_table_t table,

>                 void *key, void *value);

>

> +/**

> + * documentation missing

> + *

> + * @param table

> + * @param key

> + * @param buffer

> + * @param buffer_size

> + *

> + * @return

> + */

>  int odph_cuckoo_table_get_value(

>                 odph_table_t table,

>                 void *key, void *buffer,

>                 uint32_t buffer_size);

>

> +/**

> + * documentation missing

> + *

> + * @param table

> + * @param key

> + *

> + * @return

> + */

>  int odph_cuckoo_table_remove_value(odph_table_t table, void *key);

>

> +/** documentation missing */

>  extern odph_table_ops_t odph_cuckoo_table_ops;

>

>  #ifdef __cplusplus

> diff --git a/helper/include/odp/helper/odph_hashtable.h b/helper/include/odp/helper/odph_hashtable.h

> index bb75cb9..d3dce99 100644

> --- a/helper/include/odp/helper/odph_hashtable.h

> +++ b/helper/include/odp/helper/odph_hashtable.h

> @@ -19,17 +19,74 @@

>  extern "C" {

>  #endif

>

> +/**

> + * documentation missing

> + *

> + * @param name

> + * @param capacity

> + * @param key_size

> + * @param value_size

> + *

> + * @return

> + */

>  odph_table_t odph_hash_table_create(const char *name,

>                                     uint32_t capacity,

>                                     uint32_t key_size,

>                                     uint32_t value_size);

> +

> +/**

> + * documentation missing

> + *

> + * @param name

> + *

> + * @return

> + */

>  odph_table_t odph_hash_table_lookup(const char *name);

> +

> +/**

> + * documentation missing

> + *

> + * @param table

> + *

> + * @return

> + */

>  int odph_hash_table_destroy(odph_table_t table);

> +

> +/**

> + * documentation missing

> + *

> + * @param table

> + * @param key

> + * @param value

> + *

> + * @return

> + */

>  int odph_hash_put_value(odph_table_t table, void *key, void *value);

> +

> +/**

> + * documentation missing

> + *

> + * @param table

> + * @param key

> + * @param buffer

> + * @param buffer_size

> + *

> + * @return

> + */

>  int odph_hash_get_value(odph_table_t table, void *key, void *buffer,

>                         uint32_t buffer_size);

> +

> +/**

> + * documentation missing

> + *

> + * @param table

> + * @param key

> + *

> + * @return

> + */

>  int odph_hash_remove_value(odph_table_t table, void *key);

>

> +/** documentation missing */

>  extern odph_table_ops_t odph_hash_table_ops;

>

>  #ifdef __cplusplus

> diff --git a/helper/include/odp/helper/odph_iplookuptable.h b/helper/include/odp/helper/odph_iplookuptable.h

> index 0ae6b37..f5da493 100644

> --- a/helper/include/odp/helper/odph_iplookuptable.h

> +++ b/helper/include/odp/helper/odph_iplookuptable.h

> @@ -24,31 +24,84 @@

>  extern "C" {

>  #endif

>

> +/** documentation missing */

>  typedef struct {

> -       uint32_t ip;

> -       uint8_t cidr;

> +       uint32_t ip;  /**< IP */

> +       uint8_t cidr; /**< CIDR */

>  } odph_iplookup_prefix_t;

>

> +/**

> + * documentation missing

> + *

> + * @param name

> + * @param ODP_IGNORED_1

> + * @param ODP_IGNORED_2

> + * @param value_size

> + *

> + * @return

> + */

>  odph_table_t odph_iplookup_table_create(

>                 const char *name,

>                 uint32_t ODP_IGNORED_1,

>                 uint32_t ODP_IGNORED_2,

>                 uint32_t value_size);

>

> +/**

> + * documentation missing

> + *

> + * @param name

> + *

> + * @return

> + */

>  odph_table_t odph_iplookup_table_lookup(const char *name);

>

> +/**

> + * documentation missing

> + *

> + * @param table

> + *

> + * @return

> + */

>  int odph_iplookup_table_destroy(odph_table_t table);

>

> +/**

> + * documentation missing

> + *

> + * @param table

> + * @param key

> + * @param value

> + *

> + * @return

> + */

>  int odph_iplookup_table_put_value(

>                 odph_table_t table, void *key, void *value);

>

> +/**

> + * documentation missing

> + *

> + * @param table

> + * @param key

> + * @param buffer

> + * @param buffer_size

> + *

> + * @return

> + */

>  int odph_iplookup_table_get_value(

>                 odph_table_t table, void *key,

>                 void *buffer, uint32_t buffer_size);

>

> +/**

> + * documentation missing

> + *

> + * @param table

> + * @param key

> + *

> + * @return

> + */

>  int odph_iplookup_table_remove_value(

>                 odph_table_t table, void *key);

>

> +/** documentation missing */

>  extern odph_table_ops_t odph_iplookup_table_ops;

>

>  #ifdef __cplusplus

> diff --git a/helper/include/odp/helper/odph_lineartable.h b/helper/include/odp/helper/odph_lineartable.h

> index 0b56b7f..c01f0b9 100644

> --- a/helper/include/odp/helper/odph_lineartable.h

> +++ b/helper/include/odp/helper/odph_lineartable.h

> @@ -20,16 +20,64 @@

>  extern "C" {

>  #endif

>

> +/**

> + * documentation missing

> + *

> + * @param name

> + * @param capacity

> + * @param ODP_IGNORED

> + * @param value_size

> + *

> + * @return

> + */

>  odph_table_t odph_linear_table_create(const char *name,

>                                       uint32_t capacity,

>                                       uint32_t ODP_IGNORED,

>                                       uint32_t value_size);

> +

> +/**

> + * documentation missing

> + *

> + * @param name

> + *

> + * @return

> + */

>  odph_table_t odph_linear_table_lookup(const char *name);

> +

> +/**

> + * documentation missing

> + *

> + * @param table

> + *

> + * @return

> + */

>  int odph_linear_table_destroy(odph_table_t table);

> +

> +/**

> + * documentation missing

> + *

> + * @param table

> + * @param key

> + * @param value

> + *

> + * @return

> + */

>  int odph_linear_put_value(odph_table_t table, void *key, void *value);

> +

> +/**

> + * documentation missing

> + *

> + * @param table

> + * @param key

> + * @param buffer

> + * @param buffer_size

> + *

> + * @return

> + */

>  int odph_linear_get_value(odph_table_t table, void *key, void *buffer,

>                           uint32_t buffer_size);

>

> +/** documentation missing */

>  extern odph_table_ops_t odph_linear_table_ops;

>

>  #ifdef __cplusplus

> --

> 2.8.1

>




-- 
Mike Holmes
Program Manager - Linaro Networking Group
Linaro.org │ Open source software for ARM SoCs
"Work should be fun and collaborative, the rest follows"
diff mbox series

Patch

diff --git a/helper/include/odp/helper/odph_cuckootable.h b/helper/include/odp/helper/odph_cuckootable.h
index d569980..65cd555 100644
--- a/helper/include/odp/helper/odph_cuckootable.h
+++ b/helper/include/odp/helper/odph_cuckootable.h
@@ -52,27 +52,79 @@ 
 extern "C" {
 #endif
 
+/**
+ * documentation missing
+ *
+ * @param name
+ * @param capacity
+ * @param key_size
+ * @param value_size
+ *
+ * @return
+ */
 odph_table_t odph_cuckoo_table_create(
 		const char *name,
 		uint32_t capacity,
 		uint32_t key_size,
 		uint32_t value_size);
 
+/**
+ * documentation missing
+ *
+ * @param name
+ *
+ * @return
+ */
 odph_table_t odph_cuckoo_table_lookup(const char *name);
 
+/**
+ * documentation missing
+ *
+ * @param table
+ *
+ * @return
+ */
 int odph_cuckoo_table_destroy(odph_table_t table);
 
+/**
+ * documentation missing
+ *
+ * @param table
+ * @param key
+ * @param value
+ *
+ * @return
+ */
 int odph_cuckoo_table_put_value(
 		odph_table_t table,
 		void *key, void *value);
 
+/**
+ * documentation missing
+ *
+ * @param table
+ * @param key
+ * @param buffer
+ * @param buffer_size
+ *
+ * @return
+ */
 int odph_cuckoo_table_get_value(
 		odph_table_t table,
 		void *key, void *buffer,
 		uint32_t buffer_size);
 
+/**
+ * documentation missing
+ *
+ * @param table
+ * @param key
+ *
+ * @return
+ */
 int odph_cuckoo_table_remove_value(odph_table_t table, void *key);
 
+/** documentation missing */
 extern odph_table_ops_t odph_cuckoo_table_ops;
 
 #ifdef __cplusplus
diff --git a/helper/include/odp/helper/odph_hashtable.h b/helper/include/odp/helper/odph_hashtable.h
index bb75cb9..d3dce99 100644
--- a/helper/include/odp/helper/odph_hashtable.h
+++ b/helper/include/odp/helper/odph_hashtable.h
@@ -19,17 +19,74 @@ 
 extern "C" {
 #endif
 
+/**
+ * documentation missing
+ *
+ * @param name
+ * @param capacity
+ * @param key_size
+ * @param value_size
+ *
+ * @return
+ */
 odph_table_t odph_hash_table_create(const char *name,
 				    uint32_t capacity,
 				    uint32_t key_size,
 				    uint32_t value_size);
+
+/**
+ * documentation missing
+ *
+ * @param name
+ *
+ * @return
+ */
 odph_table_t odph_hash_table_lookup(const char *name);
+
+/**
+ * documentation missing
+ *
+ * @param table
+ *
+ * @return
+ */
 int odph_hash_table_destroy(odph_table_t table);
+
+/**
+ * documentation missing
+ *
+ * @param table
+ * @param key
+ * @param value
+ *
+ * @return
+ */
 int odph_hash_put_value(odph_table_t table, void *key, void *value);
+
+/**
+ * documentation missing
+ *
+ * @param table
+ * @param key
+ * @param buffer
+ * @param buffer_size
+ *
+ * @return
+ */
 int odph_hash_get_value(odph_table_t table, void *key, void *buffer,
 			uint32_t buffer_size);
+
+/**
+ * documentation missing
+ *
+ * @param table
+ * @param key
+ *
+ * @return
+ */
 int odph_hash_remove_value(odph_table_t table, void *key);
 
+/** documentation missing */
 extern odph_table_ops_t odph_hash_table_ops;
 
 #ifdef __cplusplus
diff --git a/helper/include/odp/helper/odph_iplookuptable.h b/helper/include/odp/helper/odph_iplookuptable.h
index 0ae6b37..f5da493 100644
--- a/helper/include/odp/helper/odph_iplookuptable.h
+++ b/helper/include/odp/helper/odph_iplookuptable.h
@@ -24,31 +24,84 @@ 
 extern "C" {
 #endif
 
+/** documentation missing */
 typedef struct {
-	uint32_t ip;
-	uint8_t cidr;
+	uint32_t ip;  /**< IP */
+	uint8_t cidr; /**< CIDR */
 } odph_iplookup_prefix_t;
 
+/**
+ * documentation missing
+ *
+ * @param name
+ * @param ODP_IGNORED_1
+ * @param ODP_IGNORED_2
+ * @param value_size
+ *
+ * @return
+ */
 odph_table_t odph_iplookup_table_create(
 		const char *name,
 		uint32_t ODP_IGNORED_1,
 		uint32_t ODP_IGNORED_2,
 		uint32_t value_size);
 
+/**
+ * documentation missing
+ *
+ * @param name
+ *
+ * @return
+ */
 odph_table_t odph_iplookup_table_lookup(const char *name);
 
+/**
+ * documentation missing
+ *
+ * @param table
+ *
+ * @return
+ */
 int odph_iplookup_table_destroy(odph_table_t table);
 
+/**
+ * documentation missing
+ *
+ * @param table
+ * @param key
+ * @param value
+ *
+ * @return
+ */
 int odph_iplookup_table_put_value(
 		odph_table_t table, void *key, void *value);
 
+/**
+ * documentation missing
+ *
+ * @param table
+ * @param key
+ * @param buffer
+ * @param buffer_size
+ *
+ * @return
+ */
 int odph_iplookup_table_get_value(
 		odph_table_t table, void *key,
 		void *buffer, uint32_t buffer_size);
 
+/**
+ * documentation missing
+ *
+ * @param table
+ * @param key
+ *
+ * @return
+ */
 int odph_iplookup_table_remove_value(
 		odph_table_t table, void *key);
 
+/** documentation missing */
 extern odph_table_ops_t odph_iplookup_table_ops;
 
 #ifdef __cplusplus
diff --git a/helper/include/odp/helper/odph_lineartable.h b/helper/include/odp/helper/odph_lineartable.h
index 0b56b7f..c01f0b9 100644
--- a/helper/include/odp/helper/odph_lineartable.h
+++ b/helper/include/odp/helper/odph_lineartable.h
@@ -20,16 +20,64 @@ 
 extern "C" {
 #endif
 
+/**
+ * documentation missing
+ *
+ * @param name
+ * @param capacity
+ * @param ODP_IGNORED
+ * @param value_size
+ *
+ * @return
+ */
 odph_table_t odph_linear_table_create(const char *name,
 				      uint32_t capacity,
 				      uint32_t ODP_IGNORED,
 				      uint32_t value_size);
+
+/**
+ * documentation missing
+ *
+ * @param name
+ *
+ * @return
+ */
 odph_table_t odph_linear_table_lookup(const char *name);
+
+/**
+ * documentation missing
+ *
+ * @param table
+ *
+ * @return
+ */
 int odph_linear_table_destroy(odph_table_t table);
+
+/**
+ * documentation missing
+ *
+ * @param table
+ * @param key
+ * @param value
+ *
+ * @return
+ */
 int odph_linear_put_value(odph_table_t table, void *key, void *value);
+
+/**
+ * documentation missing
+ *
+ * @param table
+ * @param key
+ * @param buffer
+ * @param buffer_size
+ *
+ * @return
+ */
 int odph_linear_get_value(odph_table_t table, void *key, void *buffer,
 			  uint32_t buffer_size);
 
+/** documentation missing */
 extern odph_table_ops_t odph_linear_table_ops;
 
 #ifdef __cplusplus