Those fields are documented but use the @private tag. Perhaps that isn't supported in older versions of doxygen?
On Wed, Feb 8, 2017 at 1:46 AM, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > just: > > ./configure > make doxygen-doc > > On 8 February 2017 at 01:36, Bill Fischofer <bill.fischo...@linaro.org> > wrote: >> >> How are you generating this? When I run make doxygen-doc it's >> completely clean. Is this another doxygen version issue? >> >> There is a small typo in one of the files so I'll send a v2 to correct >> that. >> >> On Tue, Feb 7, 2017 at 9:43 AM, Maxim Uvarov <maxim.uva...@linaro.org> >> wrote: >> > Bill, >> > >> > patch is good. Can you please add description for missing functions for >> > v2: >> > >> > /opt/Linaro/odp3.git/helper/include/odp/helper/odph_cuckootable.h:136: >> > warning: Member odph_cuckoo_table_ops (variable) of group >> > odph_cuckootable is not documented. >> > /opt/Linaro/odp3.git/helper/include/odp/helper/odph_hashtable.h:101: >> > warning: Member odph_hash_table_ops (variable) of group odph_hash_table >> > is not documented. >> > /opt/Linaro/odp3.git/helper/include/odp/helper/odph_iplookuptable.h:115: >> > warning: Member odph_iplookup_table_ops (variable) of group >> > odph_iplookuptable is not documented. >> > /opt/Linaro/odp3.git/helper/include/odp/helper/odph_lineartable.h:91: >> > warning: Member odph_linear_table_ops (variable) of group >> > odph_lineartable is not documented. >> > /opt/Linaro/odp3.git/helper/include/odp/helper/odph_cuckootable.h:136: >> > warning: Member odph_cuckoo_table_ops (variable) of group >> > odph_cuckootable is not documented. >> > /opt/Linaro/odp3.git/helper/include/odp/helper/odph_hashtable.h:101: >> > warning: Member odph_hash_table_ops (variable) of group odph_hash_table >> > is not documented. >> > /opt/Linaro/odp3.git/helper/include/odp/helper/odph_iplookuptable.h:115: >> > warning: Member odph_iplookup_table_ops (variable) of group >> > odph_iplookuptable is not documented. >> > /opt/Linaro/odp3.git/helper/include/odp/helper/odph_lineartable.h:91: >> > warning: Member odph_linear_table_ops (variable) of group >> > odph_lineartable is not documented. >> > >> > >> > >> > On 02/02/17 07:49, Bill Fischofer wrote: >> >> Resolve Bug https://bugs.linaro.org/show_bug.cgi?id=2865 by adding >> >> missing doxygen documentation for helper table functions >> >> >> >> Signed-off-by: Bill Fischofer <bill.fischo...@linaro.org> >> >> --- >> >> helper/include/odp/helper/odph_cuckootable.h | 80 >> >> ++++++++++++++++++--- >> >> helper/include/odp/helper/odph_hashtable.h | 75 >> >> +++++++++++++++++++- >> >> helper/include/odp/helper/odph_iplookuptable.h | 97 >> >> ++++++++++++++++++++++---- >> >> helper/include/odp/helper/odph_lineartable.h | 65 ++++++++++++++++- >> >> helper/include/odp/helper/table.h | 10 ++- >> >> 5 files changed, 299 insertions(+), 28 deletions(-) >> >> >> >> diff --git a/helper/include/odp/helper/odph_cuckootable.h >> >> b/helper/include/odp/helper/odph_cuckootable.h >> >> index d5699807..ff0a26c3 100644 >> >> --- a/helper/include/odp/helper/odph_cuckootable.h >> >> +++ b/helper/include/odp/helper/odph_cuckootable.h >> >> @@ -52,28 +52,92 @@ >> >> extern "C" { >> >> #endif >> >> >> >> +/** >> >> + * @addtogroup odph_cuckootable ODPH CUCKOO TABLE >> >> + * @{ >> >> + */ >> >> + >> >> +/** >> >> + * Create a cuckoo table >> >> + * >> >> + * @param name Name of the cuckoo table to be created >> >> + * @param capacity Number of elements table may store >> >> + * @param key_size Size of the key for each element >> >> + * @param value_size Size of the value stored for each element >> >> + * >> >> + * @return Handle of created cuckoo table >> >> + * @retval NULL Create failed >> >> + */ >> >> odph_table_t odph_cuckoo_table_create( >> >> const char *name, >> >> uint32_t capacity, >> >> uint32_t key_size, >> >> uint32_t value_size); >> >> >> >> +/** >> >> + * Lookup a cuckoo table by name >> >> + * >> >> + * @param name Name of the table to be located >> >> + * >> >> + * @return Handle of the located cuckoo table >> >> + * @retval NULL No table matching supplied name found >> >> + */ >> >> odph_table_t odph_cuckoo_table_lookup(const char *name); >> >> >> >> +/** >> >> + * Destroy a cuckoo table >> >> + * >> >> + * @param table Handle of the cuckoo table to be destroyed >> >> + * >> >> + * @retval 0 Success >> >> + * @retval < 0 Failure >> >> + */ >> >> int odph_cuckoo_table_destroy(odph_table_t table); >> >> >> >> -int odph_cuckoo_table_put_value( >> >> - odph_table_t table, >> >> - void *key, void *value); >> >> +/** >> >> + * Insert a key/value pair into a cuckoo table >> >> + * >> >> + * @param table Table into which value is to be stored >> >> + * @param key Address of an odph_table_t to be used as key >> >> + * @param value Value to be associated with specified key >> >> + * >> >> + * @retval >= 0 Success >> >> + * @retval < 0 Failure >> >> + */ >> >> +int odph_cuckoo_table_put_value(odph_table_t table, void *key, void >> >> *value); >> >> >> >> -int odph_cuckoo_table_get_value( >> >> - odph_table_t table, >> >> - void *key, void *buffer, >> >> - uint32_t buffer_size); >> >> +/** >> >> + * Retrieve a value from a cuckoo table >> >> + * >> >> + * @param table Table from which value is to be retrieved >> >> + * @param key Address of an odph_table_t to be used as key >> >> + * @param[out] buffer Address of buffer to receive resulting value >> >> + * @param buffer_size Size of supplied buffer >> >> + * >> >> + * @retval 0 Success >> >> + * @retval 1 Success >> >> + * @retval < 0 Failure >> >> + */ >> >> +int odph_cuckoo_table_get_value(odph_table_t table, >> >> + void *key, void *buffer, >> >> + uint32_t buffer_size); >> >> >> >> +/** >> >> + * Remove a value from a cuckoo table >> >> + * >> >> + * @param table Table from which value is to be removed >> >> + * @param key Address of odph_table_t to be used as key >> >> + * >> >> + * @retval >= 0 Success >> >> + * @retval < 0 Failure >> >> + */ >> >> int odph_cuckoo_table_remove_value(odph_table_t table, void *key); >> >> >> >> -extern odph_table_ops_t odph_cuckoo_table_ops; >> >> +extern odph_table_ops_t odph_cuckoo_table_ops; /**< @private */s >> >> + >> >> +/** >> >> + * @} >> >> + */ >> >> >> >> #ifdef __cplusplus >> >> } >> >> diff --git a/helper/include/odp/helper/odph_hashtable.h >> >> b/helper/include/odp/helper/odph_hashtable.h >> >> index bb75cb9f..434c6ee5 100644 >> >> --- a/helper/include/odp/helper/odph_hashtable.h >> >> +++ b/helper/include/odp/helper/odph_hashtable.h >> >> @@ -19,22 +19,93 @@ >> >> extern "C" { >> >> #endif >> >> >> >> +/** >> >> + * @addtogroup odph_hash_table ODPH HASH TABLE >> >> + * @{ >> >> + */ >> >> + >> >> +/** >> >> + * Create a hash table >> >> + * >> >> + * @param name Name of the hash table to be created. >> >> + * @param capacity Number of elements table may store >> >> + * @param key_size Size of the key for each element >> >> + * @param value_size Size of the value stored for each element >> >> + * >> >> + * @return Handle of created hash table >> >> + * @retval NULL Create failed >> >> + */ >> >> odph_table_t odph_hash_table_create(const char *name, >> >> uint32_t capacity, >> >> uint32_t key_size, >> >> uint32_t value_size); >> >> + >> >> +/** >> >> + * Lookup a hash table by name >> >> + * >> >> + * @param name Name of the table to be located >> >> + * >> >> + * @return Handle of the located hash table >> >> + * @return NULL No table matching supplied name found >> >> + */ >> >> odph_table_t odph_hash_table_lookup(const char *name); >> >> + >> >> +/** >> >> + * Destroy a hash table >> >> + * >> >> + * @param table Handle of the hash table to be destroyed >> >> + * >> >> + * @retval 0 Success >> >> + * @retval < 0 Failure >> >> + */ >> >> int odph_hash_table_destroy(odph_table_t table); >> >> + >> >> +/** >> >> + * Insert a key/value pair into a hash table >> >> + * >> >> + * @param table Table into which value is to be stored >> >> + * @param key Address of an odph_table_t to be used as key >> >> + * @param value Value to be associated with specified key >> >> + * >> >> + * @retval >= 0 Success >> >> + * @retval < 0 Failure >> >> + */ >> >> int odph_hash_put_value(odph_table_t table, void *key, void *value); >> >> + >> >> +/** >> >> + * Retrieve a value from a hash table >> >> + * >> >> + * @param table Table from which value is to be retrieved >> >> + * @param key Address of an odph_table_t to be used as key >> >> + * @param[out] buffer Address of buffer to receive resulting value >> >> + * @param buffer_size Size of supplied buffer >> >> + * >> >> + * @retval 0 Success >> >> + * @retval 1 Success >> >> + * @retval < 0 Failure >> >> + */ >> >> int odph_hash_get_value(odph_table_t table, void *key, void *buffer, >> >> uint32_t buffer_size); >> >> + >> >> +/** >> >> + * Remove a value from a hash table >> >> + * >> >> + * @param table Table from which value is to be removed >> >> + * @param key Address of odph_table_t to be used as key >> >> + * >> >> + * @retval >= 0 Success >> >> + * @retval < 0 Failure >> >> + */ >> >> int odph_hash_remove_value(odph_table_t table, void *key); >> >> >> >> -extern odph_table_ops_t odph_hash_table_ops; >> >> +extern odph_table_ops_t odph_hash_table_ops; /**< @private */ >> >> + >> >> +/** >> >> + * @} >> >> + */ >> >> >> >> #ifdef __cplusplus >> >> } >> >> #endif >> >> >> >> #endif >> >> - >> >> diff --git a/helper/include/odp/helper/odph_iplookuptable.h >> >> b/helper/include/odp/helper/odph_iplookuptable.h >> >> index 0ae6b376..7687675c 100644 >> >> --- a/helper/include/odp/helper/odph_iplookuptable.h >> >> +++ b/helper/include/odp/helper/odph_iplookuptable.h >> >> @@ -24,32 +24,99 @@ >> >> extern "C" { >> >> #endif >> >> >> >> +/** >> >> + * @addtogroup odph_iplookuptable ODPH IP LOOKUP TABLE >> >> + * @{ >> >> + */ >> >> + >> >> +/** >> >> + * IP Lookup Prefix >> >> + */ >> >> typedef struct { >> >> - uint32_t ip; >> >> - uint8_t cidr; >> >> + uint32_t ip; /**< IPv4 address */ >> >> + uint8_t cidr; /**< CIDR value for prefix matching */ >> >> } odph_iplookup_prefix_t; >> >> >> >> -odph_table_t odph_iplookup_table_create( >> >> - const char *name, >> >> - uint32_t ODP_IGNORED_1, >> >> - uint32_t ODP_IGNORED_2, >> >> - uint32_t value_size); >> >> +/** >> >> + * Create an IP lookup table >> >> + * >> >> + * @param name Name of the table to be created >> >> + * @param ODP_IGNORED_1 Unused >> >> + * @param ODP_IGNORED_2 Unused >> >> + * @param value_size Byte size of each entry in the table >> >> + * >> >> + * @return Handle of the created ip lookup table >> >> + * @retval NULL If table create failed >> >> + */ >> >> +odph_table_t odph_iplookup_table_create(const char *name, >> >> + uint32_t ODP_IGNORED_1, >> >> + uint32_t ODP_IGNORED_2, >> >> + uint32_t value_size); >> >> >> >> +/** >> >> + * Lookup an IP lookup table by name >> >> + * >> >> + * @param name Name of the table to be located >> >> + * >> >> + * @return Handle of the located ip lookup table >> >> + * @retval NULL No table matching supplied name found >> >> + */ >> >> odph_table_t odph_iplookup_table_lookup(const char *name); >> >> >> >> +/** >> >> + * Destroy an IP lookup table >> >> + * >> >> + * @param table Handle of the ip lookup table to be destroyed >> >> + * >> >> + * @retval 0 Success >> >> + * @retval < 0 Failure >> >> + */ >> >> int odph_iplookup_table_destroy(odph_table_t table); >> >> >> >> -int odph_iplookup_table_put_value( >> >> - odph_table_t table, void *key, void *value); >> >> +/** >> >> + * Insert a key/value pair into an ip lookup table >> >> + * >> >> + * @param table Table into which value is to be stored >> >> + * @param key Address of an odph_iplookup_prefix_t to be used as key >> >> + * @param value Value to be associated with specified key >> >> + * >> >> + * @retval >= 0 Success >> >> + * @retval < 0 Failure >> >> + */ >> >> +int odph_iplookup_table_put_value(odph_table_t table, void *key, void >> >> *value); >> >> + >> >> +/** >> >> + * Retrieve a value from an iplookup table >> >> + * >> >> + * @param table Table from which value is to be retrieved >> >> + * @param key Address of an odph_iplookup_prefix_t to be used as key >> >> + * @param[out] buffer Address of buffer to receive resulting value >> >> + * @param buffer_size Size of supplied buffer >> >> + * >> >> + * @retval 0 Success >> >> + * @retval 1 Success >> >> + * @retval < 0 Failure >> >> + */ >> >> +int odph_iplookup_table_get_value(odph_table_t table, void *key, >> >> + void *buffer, uint32_t buffer_size); >> >> >> >> -int odph_iplookup_table_get_value( >> >> - odph_table_t table, void *key, >> >> - void *buffer, uint32_t buffer_size); >> >> +/** >> >> + * Remove a value from an iplookup table >> >> + * >> >> + * @param table Table from which value is to be removed >> >> + * @param key Address of odph_iplookup_prefix_t to be used as key >> >> + * >> >> + * @retval >= 0 Success >> >> + * @retval < 0 Failure >> >> + * >> >> + */ >> >> +int odph_iplookup_table_remove_value(odph_table_t table, void *key); >> >> >> >> -int odph_iplookup_table_remove_value( >> >> - odph_table_t table, void *key); >> >> +extern odph_table_ops_t odph_iplookup_table_ops; /**< @private */ >> >> >> >> -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 0b56b7fa..10874d8c 100644 >> >> --- a/helper/include/odp/helper/odph_lineartable.h >> >> +++ b/helper/include/odp/helper/odph_lineartable.h >> >> @@ -20,21 +20,82 @@ >> >> extern "C" { >> >> #endif >> >> >> >> +/** >> >> + * @addtogroup odph_lineartable ODPH LINEAR TABLE >> >> + * @{ >> >> + */ >> >> + >> >> +/** >> >> + * Create a linear table >> >> + * >> >> + * @param name Name of the linear table to be created >> >> + * @param capacity Number of elements table may store >> >> + * @param ODP_IGNORED Ignored parameter >> >> + * @param value_size Size of the value stored for each element >> >> + * >> >> + * @return Handle of created linear table >> >> + * @return NULL Create failed >> >> + */ >> >> odph_table_t odph_linear_table_create(const char *name, >> >> uint32_t capacity, >> >> uint32_t ODP_IGNORED, >> >> uint32_t value_size); >> >> + >> >> +/** >> >> + * Lookup a linear table >> >> + * >> >> + * @param name Name of the table to be located >> >> + * >> >> + * @return Handle of the located linear table >> >> + * @retval NULL No table matching supplied name found >> >> + */ >> >> odph_table_t odph_linear_table_lookup(const char *name); >> >> + >> >> +/** >> >> + * Destroy a linear table >> >> + * >> >> + * @param table Handle of linear table to be destroyed >> >> + * >> >> + * @retval 0 Success >> >> + * @retval < 0 Failure >> >> + */ >> >> int odph_linear_table_destroy(odph_table_t table); >> >> + >> >> +/** >> >> + * Insert a value into a linear table >> >> + * >> >> + * @param table Table into which value is to be stored >> >> + * @param key Index value used as key >> >> + * @param value Value to be assoceiated with specified key index >> >> + * >> >> + * @retval >= 0 Success >> >> + * @retval < 0 Failure >> >> + */ >> >> int odph_linear_put_value(odph_table_t table, void *key, void *value); >> >> + >> >> +/** >> >> + * Retrieve a value from a linear table >> >> + * >> >> + * @param table Table from which value is to be retrieved >> >> + * @param key Index value used as key >> >> + * @param[out] buffer Address of buffer to receive resulting value >> >> + * @param buffer_size Size of supplied buffer >> >> + * >> >> + * @retval 0 Success >> >> + * @retval 1 Success >> >> + * @retval < 0 Failure >> >> + */ >> >> int odph_linear_get_value(odph_table_t table, void *key, void *buffer, >> >> uint32_t buffer_size); >> >> >> >> -extern odph_table_ops_t odph_linear_table_ops; >> >> +extern odph_table_ops_t odph_linear_table_ops; /**< @private */ >> >> + >> >> +/** >> >> + * @} >> >> + */ >> >> >> >> #ifdef __cplusplus >> >> } >> >> #endif >> >> >> >> #endif >> >> - >> >> diff --git a/helper/include/odp/helper/table.h >> >> b/helper/include/odp/helper/table.h >> >> index 81022e55..b3440ef5 100644 >> >> --- a/helper/include/odp/helper/table.h >> >> +++ b/helper/include/odp/helper/table.h >> >> @@ -80,6 +80,11 @@ >> >> extern "C" { >> >> #endif >> >> >> >> +/** >> >> + * @addtogroup odph_tables ODPH TABLES >> >> + * @{ >> >> + */ >> >> + >> >> #include <stdint.h> >> >> >> >> /** >> >> @@ -228,9 +233,12 @@ typedef struct odph_table_ops_t { >> >> odph_table_remove_value f_remove; >> >> } odph_table_ops_t; >> >> >> >> +/** >> >> + * @} >> >> + */ >> >> + >> >> #ifdef __cplusplus >> >> } >> >> #endif >> >> >> >> #endif >> >> - >> >> >> > > >