[dpdk-dev] [PATCH v1 1/5] devtools: script to remove unused headers includes
Bruce Richardson
bruce.richardson at intel.com
Wed Oct 6 18:28:06 CEST 2021
On Wed, Oct 06, 2021 at 04:19:46PM +0100, Morrissey, Sean wrote:
>
> On 06/10/2021 14:37, Thomas Monjalon wrote:
> > 04/10/2021 12:10, Sean Morrissey:
> > > This script can be used for removing headers flagged for removal by the
> > > include-what-you-use (IWYU) tool. The script has the ability to remove
> > > headers from specified sub-directories or dpdk as a whole.
> > My fear is that it could flag an include as useless,
> > while it is required for a different system (BSD, Windows).
> >
> >
> Hi Thomas,
>
> Stephen raised a similar concern. Please see Bruce's reply here: https://patches.dpdk.org/project/dpdk/cover/20211004101058.2396458-1-sean.morrissey@intel.com/
>
One additional data point. Checking the output of a run I did of IWYU a
month or two ago, shows 2254 header includes that IWYU thinks should be
removed. With that sort of scale of an issue, I think automation will be
needed.
/Bruce
More information about the dev
mailing list