From: "Philippe Mathieu-Daudé" <philmd@linaro.org>
To: qemu-devel@nongnu.org
Cc: qemu-s390x@nongnu.org, qemu-ppc@nongnu.org,
qemu-riscv@nongnu.org, qemu-arm@nongnu.org,
"Philippe Mathieu-Daudé" <philmd@linaro.org>
Subject: [PATCH 0/5] misc: Enforce .[ch].inc extension for re-included .c/.h files
Date: Tue, 6 Jun 2023 16:12:47 +0200 [thread overview]
Message-ID: <20230606141252.95032-1-philmd@linaro.org> (raw)
Follow the convention to use the .inc extension for .c/.h files
re-included, as docummented in Coding Style since commit 6a0057aa22:
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.
Most of the codebase already uses this extension:
$ git ls-files | fgrep .inc | wc -l
155
Rename some files left over.
Philippe Mathieu-Daudé (5):
bsd-user: Rename elfcore.c -> elfcore.c.inc
target/arm: Rename helper template headers as '.h.inc'
target/i386: Rename helper template headers as '.h.inc'
target: Rename per-target 'helper.h' -> 'helper.h.inc'
tests/tcg/aarch64: Rename bti-crt.inc.c -> bti-crt.c.inc
docs/devel/tcg-ops.rst | 2 +-
include/exec/helper-gen.h | 2 +-
include/exec/helper-proto.h | 2 +-
target/alpha/{helper.h => helper.h.inc} | 0
target/arm/{helper.h => helper.h.inc} | 8 ++++----
target/arm/tcg/{helper-a64.h => helper-a64.h.inc} | 0
target/arm/tcg/{helper-mve.h => helper-mve.h.inc} | 0
target/arm/tcg/{helper-sme.h => helper-sme.h.inc} | 0
target/arm/tcg/{helper-sve.h => helper-sve.h.inc} | 0
target/avr/{helper.h => helper.h.inc} | 0
target/cris/{helper.h => helper.h.inc} | 0
target/hexagon/{helper.h => helper.h.inc} | 0
target/hppa/{helper.h => helper.h.inc} | 0
target/i386/{helper.h => helper.h.inc} | 6 +++---
target/i386/{ops_sse_header.h => ops_sse_header.h.inc} | 0
...hift_helper_template.h => shift_helper_template.h.inc} | 0
.../{cc_helper_template.h => cc_helper_template.h.inc} | 0
target/loongarch/{helper.h => helper.h.inc} | 0
target/m68k/{helper.h => helper.h.inc} | 0
target/microblaze/{helper.h => helper.h.inc} | 0
target/mips/{helper.h => helper.h.inc} | 0
target/nios2/{helper.h => helper.h.inc} | 0
target/openrisc/{helper.h => helper.h.inc} | 0
target/ppc/{helper.h => helper.h.inc} | 0
target/riscv/{helper.h => helper.h.inc} | 0
target/rx/{helper.h => helper.h.inc} | 0
target/s390x/{helper.h => helper.h.inc} | 0
target/sh4/{helper.h => helper.h.inc} | 0
target/sparc/{helper.h => helper.h.inc} | 0
target/tricore/{helper.h => helper.h.inc} | 0
target/xtensa/{helper.h => helper.h.inc} | 0
bsd-user/elfload.c | 2 +-
target/alpha/translate.c | 2 +-
target/arm/machine.c | 2 +-
target/arm/tcg/translate.c | 2 +-
target/avr/translate.c | 2 +-
target/cris/translate.c | 2 +-
target/hexagon/translate.c | 2 +-
target/hppa/translate.c | 2 +-
target/i386/tcg/cc_helper.c | 8 ++++----
target/i386/tcg/int_helper.c | 8 ++++----
target/i386/tcg/translate.c | 2 +-
target/loongarch/translate.c | 2 +-
target/m68k/translate.c | 2 +-
target/microblaze/translate.c | 2 +-
target/mips/tcg/translate.c | 2 +-
target/nios2/translate.c | 2 +-
target/openrisc/translate.c | 2 +-
target/ppc/translate.c | 2 +-
target/riscv/translate.c | 2 +-
target/rx/translate.c | 2 +-
target/s390x/tcg/translate.c | 2 +-
target/sh4/translate.c | 2 +-
target/sparc/translate.c | 2 +-
target/tricore/translate.c | 2 +-
target/xtensa/translate.c | 2 +-
tests/tcg/aarch64/bti-1.c | 2 +-
tests/tcg/aarch64/bti-3.c | 2 +-
bsd-user/{elfcore.c => elfcore.c.inc} | 0
tests/tcg/aarch64/{bti-crt.inc.c => bti-crt.c.inc} | 0
60 files changed, 43 insertions(+), 43 deletions(-)
rename target/alpha/{helper.h => helper.h.inc} (100%)
rename target/arm/{helper.h => helper.h.inc} (99%)
rename target/arm/tcg/{helper-a64.h => helper-a64.h.inc} (100%)
rename target/arm/tcg/{helper-mve.h => helper-mve.h.inc} (100%)
rename target/arm/tcg/{helper-sme.h => helper-sme.h.inc} (100%)
rename target/arm/tcg/{helper-sve.h => helper-sve.h.inc} (100%)
rename target/avr/{helper.h => helper.h.inc} (100%)
rename target/cris/{helper.h => helper.h.inc} (100%)
rename target/hexagon/{helper.h => helper.h.inc} (100%)
rename target/hppa/{helper.h => helper.h.inc} (100%)
rename target/i386/{helper.h => helper.h.inc} (98%)
rename target/i386/{ops_sse_header.h => ops_sse_header.h.inc} (100%)
rename target/i386/{shift_helper_template.h => shift_helper_template.h.inc} (100%)
rename target/i386/tcg/{cc_helper_template.h => cc_helper_template.h.inc} (100%)
rename target/loongarch/{helper.h => helper.h.inc} (100%)
rename target/m68k/{helper.h => helper.h.inc} (100%)
rename target/microblaze/{helper.h => helper.h.inc} (100%)
rename target/mips/{helper.h => helper.h.inc} (100%)
rename target/nios2/{helper.h => helper.h.inc} (100%)
rename target/openrisc/{helper.h => helper.h.inc} (100%)
rename target/ppc/{helper.h => helper.h.inc} (100%)
rename target/riscv/{helper.h => helper.h.inc} (100%)
rename target/rx/{helper.h => helper.h.inc} (100%)
rename target/s390x/{helper.h => helper.h.inc} (100%)
rename target/sh4/{helper.h => helper.h.inc} (100%)
rename target/sparc/{helper.h => helper.h.inc} (100%)
rename target/tricore/{helper.h => helper.h.inc} (100%)
rename target/xtensa/{helper.h => helper.h.inc} (100%)
rename bsd-user/{elfcore.c => elfcore.c.inc} (100%)
rename tests/tcg/aarch64/{bti-crt.inc.c => bti-crt.c.inc} (100%)
--
2.38.1
next reply other threads:[~2023-06-06 14:14 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-06 14:12 Philippe Mathieu-Daudé [this message]
2023-06-06 14:12 ` [PATCH 1/5] bsd-user: Rename elfcore.c -> elfcore.c.inc Philippe Mathieu-Daudé
2023-06-06 14:29 ` Richard Henderson
2023-06-06 18:08 ` Warner Losh
2023-06-06 19:08 ` Philippe Mathieu-Daudé
2023-06-06 14:12 ` [PATCH 2/5] target/arm: Rename helper template headers as '.h.inc' Philippe Mathieu-Daudé
2023-06-06 14:37 ` Richard Henderson
2023-06-06 15:49 ` Philippe Mathieu-Daudé
2023-06-06 15:55 ` Peter Maydell
2023-06-06 16:31 ` Richard Henderson
2023-06-06 14:12 ` [PATCH 3/5] target/i386: " Philippe Mathieu-Daudé
2023-06-06 14:38 ` Richard Henderson
2023-06-06 14:12 ` [PATCH 4/5] target: Rename per-target 'helper.h' -> 'helper.h.inc' Philippe Mathieu-Daudé
2023-06-06 14:43 ` Richard Henderson
2023-06-08 6:46 ` Philippe Mathieu-Daudé
2023-06-06 14:12 ` [PATCH 5/5] tests/tcg/aarch64: Rename bti-crt.inc.c -> bti-crt.c.inc Philippe Mathieu-Daudé
2023-06-06 14:29 ` Richard Henderson
2023-06-06 17:05 ` Alex Bennée
2023-06-06 17:02 ` [PATCH 0/5] misc: Enforce .[ch].inc extension for re-included .c/.h files Alex Bennée
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230606141252.95032-1-philmd@linaro.org \
--to=philmd@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=qemu-riscv@nongnu.org \
--cc=qemu-s390x@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).