Re: [lng-odp] [PATCH] doc: helper: add missing doxygen for helper table functions

2017-02-08 Thread Bill Fischofer
I'm on 1.8.11 (standard with Ubuntu 16.10) On Wed, Feb 8, 2017 at 6:53 AM, Maxim Uvarov 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.

Re: [lng-odp] [PATCH] doc: helper: add missing doxygen for helper table functions

2017-02-08 Thread Maxim Uvarov
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:4

Re: [lng-odp] [PATCH] doc: helper: add missing doxygen for helper table functions

2017-02-08 Thread Bill Fischofer
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 wrote: > just: > > ./configure > make doxygen-doc > > On 8 February 2017 at 01:36, Bill Fischofer > wrote: >> >> How are you generating th

Re: [lng-odp] [PATCH] doc: helper: add missing doxygen for helper table functions

2017-02-07 Thread Maxim Uvarov
just: ./configure make doxygen-doc On 8 February 2017 at 01:36, Bill Fischofer 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. > >

Re: [lng-odp] [PATCH] doc: helper: add missing doxygen for helper table functions

2017-02-07 Thread Bill Fischofer
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 wrote: > Bill, > > patch is good. Can you please add de

Re: [lng-odp] [PATCH] doc: helper: add missing doxygen for helper table functions

2017-02-07 Thread Maxim Uvarov
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/odp

[lng-odp] [PATCH] doc: helper: add missing doxygen for helper table functions

2017-02-01 Thread Bill Fischofer
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 --- helper/include/odp/helper/odph_cuckootable.h | 80 ++--- helper/include/odp/helper/odph_hashtable.h | 75 +++