I'm on 1.8.11 (standard with Ubuntu 16.10)
On Wed, Feb 8, 2017 at 6:53 AM, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > On 02/08/17 15:48, Bill Fischofer wrote: >> Those fields are documented but use the @private tag. Perhaps that >> isn't supported in older versions of doxygen? >> > > mine is: > doxygen --version > 1.8.6 > > DEPENDENCIES says 1.8.8 is supported version. > > which version do you use? > > >> 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 >>>>>> - >>>>>> >>>>> >>> >>> >