<snip> > Subject: Re: [PATCH v2 12/12] app: remove unnecessary include of atomic > header file > > On Tue, Nov 16, 2021 at 10:44 AM Joyce Kong <joyce.k...@arm.com> wrote: > > > > Remove the unnecessary rte_atomic.h included in app modules. > > > > Signed-off-by: Joyce Kong <joyce.k...@arm.com> > > Reviewed-by: Ruifeng Wang <ruifeng.w...@arm.com> > > After patch, I still see: > > $ git grep rte_atomic.h app/ > app/test/commands.c:#include <rte_atomic.h> > app/test/test_atomic.c:#include <rte_atomic.h> > app/test/test_event_timer_adapter.c:#include <rte_atomic.h> > > I can undertand why the test_atomic would depend on rte_atomic.h :-) but > not the rest. > Is there a reason? or is it just a miss? > > -- > David Marchand
Hi David, I checked the rest and it was a miss. Thanks for the remind, would update in v3. Joyce
