|
From: | Alex Bennée |
Subject: | Re: [PATCH 5/5] tests/tcg/aarch64: Rename bti-crt.inc.c -> bti-crt.c.inc |
Date: | Tue, 06 Jun 2023 18:05:20 +0100 |
User-agent: | mu4e 1.11.6; emacs 29.0.91 |
Philippe Mathieu-Daudé <philmd@linaro.org> writes: > Since commit 139c1837db ("meson: rename included C source files > to .c.inc"), QEMU standard procedure for included C files is to > use *.c.inc. > > Besides, since commit 6a0057aa22 ("docs/devel: make a statement > about includes") this is documented as the Coding Style: > > If you do use template header files they should be named with > the ``.c.inc`` or ``.h.inc`` suffix to make it clear they are > being included for expansion. > > Therefore rename 'bti-crt.inc.c' as 'bti-crt.c.inc'. > > Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> Reviewed-by: Alex Bennée <alex.bennee@linaro.org> -- Alex Bennée Virtualisation Tech Lead @ Linaro
[Prev in Thread] | Current Thread | [Next in Thread] |