On Fri, Apr 7, 2017 at 9:41 AM, Peltonen, Janne (Nokia - FI/Espoo) <janne.pelto...@nokia.com> wrote: > I think it is a good suggestion. But I understood that in C++ > the comma is not allowed until C++11 so I wonder if it is > ok to add it in ODP (in C it is ok from C99 on).
Given that ODP "accommodates" C++ rather than supports it, I'd think it's reasonable to say we require C++11 at this point, given that C++14 is the current standard and C++17 is being finalized. > > Perhaps someone can just add the comma when pushing the patch? > > Janne > >> -----Original Message----- >> From: Bill Fischofer [mailto:bill.fischo...@linaro.org] >> Sent: Friday, April 07, 2017 3:53 PM >> To: Peltonen, Janne (Nokia - FI/Espoo) <janne.pelto...@nokia.com> >> Cc: lng-odp-forward <lng-odp@lists.linaro.org> >> Subject: Re: [lng-odp] [API-NEXT PATCH] abi: event: add >> ODP_EVENT_IPSEC_STATUS >> >> On Fri, Apr 7, 2017 at 4:54 AM, Janne Peltonen <janne.pelto...@nokia.com> >> wrote: >> > >> > Update ABI spec with the new IPsec event type. >> > >> > Signed-off-by: Janne Peltonen <janne.pelto...@nokia.com> >> >> Reviewed-by: Bill Fischofer <bill.fischo...@linaro.org> >> >> > --- >> > include/odp/arch/default/api/abi/event.h | 3 ++- >> > 1 file changed, 2 insertions(+), 1 deletion(-) >> > >> > diff --git a/include/odp/arch/default/api/abi/event.h >> b/include/odp/arch/default/api/abi/event.h >> > index ab14157..87220d6 100644 >> > --- a/include/odp/arch/default/api/abi/event.h >> > +++ b/include/odp/arch/default/api/abi/event.h >> > @@ -29,7 +29,8 @@ typedef enum odp_event_type_t { >> > ODP_EVENT_PACKET = 2, >> > ODP_EVENT_TIMEOUT = 3, >> > ODP_EVENT_CRYPTO_COMPL = 4, >> > - ODP_EVENT_IPSEC_RESULT = 5 >> > + ODP_EVENT_IPSEC_RESULT = 5, >> > + ODP_EVENT_IPSEC_STATUS = 6 >> >> It's perfectly legal C for the last element in an enum to be >> comma-terminated and that makes it easier to keep extending the enum >> with additional entries since the previous entry doesn't have to keep >> being modified to add a comma, so I'd suggest adopting that convention >> here. Other than that nit, this looks fine. >> >> > } odp_event_type_t; >> > >> > /** >> > -- >> > 2.5.0 >> >