On Mon, Mar 04, 2024 at 04:35:41PM +0100, Thomas Monjalon wrote: > 21/02/2024 11:32, Bruce Richardson: > > The event vector struct was missing comments on two members, and also > > was inadvertently creating a local variable called "__rte_aligned" in > > the doxygen output. > > > > Correct the comment markers to fix the former issue, and fix the latter > > by putting "#ifdef __DOXYGEN" around the alignment constraint. > [..] > > +#ifndef __DOXYGEN__ > > } __rte_aligned(16); > > +#else > > +}; > > +#endif > > Would it be possible to make __rte_aligned empty in rte_common.h > instead of each call? Does it fix Doxygen bug? > I think that should be fixed globally by Tyler's series for "alignas"[1] With the new placement for the alignment macro, I don't think this doxygen issue will occur again.
[1] https://patches.dpdk.org/project/dpdk/list/?series=31229