[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 7/7] tests/style: check qemu/osdep.h is included in all .c
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH v2 7/7] tests/style: check qemu/osdep.h is included in all .c files |
Date: |
Mon, 4 Jul 2022 16:50:42 +0100 |
User-agent: |
Mutt/2.2.6 (2022-06-05) |
On Mon, Jul 04, 2022 at 04:47:16PM +0100, Peter Maydell wrote:
> On Mon, 4 Jul 2022 at 16:23, Daniel P. Berrangé <berrange@redhat.com> wrote:
> >
> > Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
>
> > +
> > +sc_c_file_osdep_h:
> > + @require='#include "qemu/osdep.h"' \
> > + in_vc_files='\.c$$' \
> > + halt='all C files must include qemu/osdep.h' \
> > + $(_sc_search_regexp)
>
> The rule is not just "included in all C files", but "included
> as the *first* include in all C files".
Oh right, so we can copy a rule from libvirt to validate that.
It would look like this, but s,config.h,qemu/osdep.h,
# Print each file name for which the first #include does not match
# $(config_h_header). Like grep -m 1, this only looks at the first match.
perl_config_h_first_ = \
-e 'BEGIN {$$ret = 0}' \
-e 'if (/^\# *include\b/) {' \
-e ' if (not m{^\# *include $(config_h_header)}) {' \
-e ' print "$$ARGV\n";' \
-e ' $$ret = 1;' \
-e ' }' \
-e ' \# Move on to next file after first include' \
-e ' close ARGV;' \
-e '}' \
-e 'END {exit $$ret}'
# You must include <config.h> before including any other header file.
# This can possibly be via a package-specific header, if given by
syntax-check.mk.
sc_require_config_h_first:
@if $(VC_LIST_EXCEPT) | $(GREP) '\.c$$' > /dev/null; then \
files=$$($(VC_LIST_EXCEPT) | $(GREP) '\.c$$') && \
perl -n $(perl_config_h_first_) $$files || \
{ echo 'the above files include some other header' \
'before <config.h>' 1>&2; exit 1; } || :; \
else :; \
fi
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
[PATCH v2 6/7] misc: ensure qemu/osdep.h is included in all .c files, Daniel P . Berrangé, 2022/07/04