For the entire patch set: Reviewed-by: Petri Savolainen <petri.savolai...@nokia.com>
> -----Original Message----- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of > Matias Elo > Sent: Tuesday, December 20, 2016 3:45 PM > To: lng-odp@lists.linaro.org > Subject: Suspected SPAM - [lng-odp] [API-NEXT PATCH 1/3] api: unify > ODP_*_NAME_LEN specifications > > Unify name length definitions to always include terminating null > character. > > Signed-off-by: Matias Elo <matias....@nokia.com> > --- > include/odp/api/spec/classification.h | 2 +- > include/odp/api/spec/pool.h | 2 +- > include/odp/api/spec/queue.h | 2 +- > include/odp/api/spec/schedule.h | 2 +- > include/odp/api/spec/shared_memory.h | 2 +- > include/odp/api/spec/timer.h | 2 +- > 6 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/include/odp/api/spec/classification.h > b/include/odp/api/spec/classification.h > index 0e442c7..0e1addd 100644 > --- a/include/odp/api/spec/classification.h > +++ b/include/odp/api/spec/classification.h > @@ -44,7 +44,7 @@ extern "C" { > > /** > * @def ODP_COS_NAME_LEN > - * Maximum ClassOfService name length in chars > + * Maximum ClassOfService name length in chars including null char > */ > > /** > diff --git a/include/odp/api/spec/pool.h b/include/odp/api/spec/pool.h > index 041f4af..af2829b 100644 > --- a/include/odp/api/spec/pool.h > +++ b/include/odp/api/spec/pool.h > @@ -36,7 +36,7 @@ extern "C" { > * Invalid pool > */ > > -/** Maximum queue name length in chars */ > +/** Maximum pool name length in chars including null char */ > #define ODP_POOL_NAME_LEN 32 > > /** > diff --git a/include/odp/api/spec/queue.h b/include/odp/api/spec/queue.h > index b0c5e31..7972fea 100644 > --- a/include/odp/api/spec/queue.h > +++ b/include/odp/api/spec/queue.h > @@ -44,7 +44,7 @@ extern "C" { > > /** > * @def ODP_QUEUE_NAME_LEN > - * Maximum queue name length in chars > + * Maximum queue name length in chars including null char > */ > > /** > diff --git a/include/odp/api/spec/schedule.h > b/include/odp/api/spec/schedule.h > index f976a4c..8244746 100644 > --- a/include/odp/api/spec/schedule.h > +++ b/include/odp/api/spec/schedule.h > @@ -42,7 +42,7 @@ extern "C" { > > /** > * @def ODP_SCHED_GROUP_NAME_LEN > - * Maximum schedule group name length in chars > + * Maximum schedule group name length in chars including null char > */ > > /** > diff --git a/include/odp/api/spec/shared_memory.h > b/include/odp/api/spec/shared_memory.h > index 885751d..074c883 100644 > --- a/include/odp/api/spec/shared_memory.h > +++ b/include/odp/api/spec/shared_memory.h > @@ -40,7 +40,7 @@ extern "C" { > * Synonym for buffer pool use > */ > > -/** Maximum shared memory block name length in chars */ > +/** Maximum shared memory block name length in chars including null char > */ > #define ODP_SHM_NAME_LEN 32 > > /* > diff --git a/include/odp/api/spec/timer.h b/include/odp/api/spec/timer.h > index 49221c4..46a4369 100644 > --- a/include/odp/api/spec/timer.h > +++ b/include/odp/api/spec/timer.h > @@ -90,7 +90,7 @@ typedef enum { > ODP_TIMER_NOEVENT = -3 > } odp_timer_set_t; > > -/** Maximum timer pool name length in chars (including null char) */ > +/** Maximum timer pool name length in chars including null char */ > #define ODP_TIMER_POOL_NAME_LEN 32 > > /** Timer pool parameters > -- > 2.7.4
