Re: [PATCH v3 1/4] kni: add define guards to avoid multi-inclusion

2022-02-22 Thread Thomas Monjalon
09/02/2022 14:52, Ferruh Yigit: > On 2/9/2022 7:24 AM, Weiguo Li wrote: > > +#ifndef RTE_KNI_FIFO_H > > +#define RTE_KNI_FIFO_H > > It doesn't really differ much but other kni header guards wrapped with '_', > I don't now why. No good reason I think. > If there will be a new version can you plea

Re: [PATCH v3 1/4] kni: add define guards to avoid multi-inclusion

2022-02-09 Thread Ferruh Yigit
On 2/9/2022 7:24 AM, Weiguo Li wrote: Supplement define guards to prevent multiple inclusion. It is kind of private header and only 'rte_kni.c' includes it, that is why missing guards doesn't cause trouble but no harm/objection to have them. Signed-off-by: Weiguo Li Acked-by: Ferruh Yigit