[LTP] [PATCH] tools: Check headers with checkpatch.pl

Petr Vorel pvorel@suse.cz
Tue Jul 26 16:05:08 CEST 2022


...
> > LGTM, just a small nit: target for headers contain .h extension.
> > Shouldn't we add .c for C sources for consistency?

> > It'd strange, that make check in testcases/misc/math/float/ does not include
> > tfloat.h, but make check-tfloat.h prints errors. But make check in lib/ works as
> > expected (adds also *.h).

> The Sparse based tool does include headers though. So really we are
> checking the target which includes more than just a single c file.

Ah, that makes sense then. Thanks!

Kind regards,
Petr


More information about the ltp mailing list