mbox series

[0/2] misc: Rename included template headers using '.inc' suffix

Message ID 20240910112833.27594-1-philmd@linaro.org (mailing list archive)
Headers show
Series misc: Rename included template headers using '.inc' suffix | expand

Message

Philippe Mathieu-Daudé Sept. 10, 2024, 11:28 a.m. UTC
Follow 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.

Philippe Mathieu-Daudé (2):
  target/hexagon: Rename macros.inc -> macros.h.inc
  tests/bench: Rename test_akcipher_keys.inc -> test_akcipher_keys.c.inc

 target/hexagon/idef-parser/README.rst                         | 4 ++--
 target/hexagon/idef-parser/{macros.inc => macros.h.inc}       | 0
 tests/bench/benchmark-crypto-akcipher.c                       | 2 +-
 .../{test_akcipher_keys.inc => test_akcipher_keys.c.inc}      | 0
 target/hexagon/gen_idef_parser_funcs.py                       | 2 +-
 target/hexagon/meson.build                                    | 2 +-
 6 files changed, 5 insertions(+), 5 deletions(-)
 rename target/hexagon/idef-parser/{macros.inc => macros.h.inc} (100%)
 rename tests/bench/{test_akcipher_keys.inc => test_akcipher_keys.c.inc} (100%)

Comments

Richard Henderson Sept. 10, 2024, 7:18 p.m. UTC | #1
On 9/10/24 04:28, Philippe Mathieu-Daudé wrote:
> Philippe Mathieu-Daudé (2):
>    target/hexagon: Rename macros.inc -> macros.h.inc
>    tests/bench: Rename test_akcipher_keys.inc -> test_akcipher_keys.c.inc

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>


r~
Michael Tokarev Sept. 20, 2024, 7:09 a.m. UTC | #2
On 10.09.2024 14:28, Philippe Mathieu-Daudé wrote:
> Follow 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.
> 
> Philippe Mathieu-Daudé (2):
>    target/hexagon: Rename macros.inc -> macros.h.inc
>    tests/bench: Rename test_akcipher_keys.inc -> test_akcipher_keys.c.inc

Picked up for the trivial-patches tree, thanks!

/mjt