misc: Rename included template headers using '.inc' suffix

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

Philippe Mathieu-Daudé posted 2 patches 1 year, 5 months ago

Download series mbox

Failed in applying to current master (apply log)

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%)

Expand all Fold all

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

Posted by Philippe Mathieu-Daudé 1 year, 5 months ago

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%)

-- 
2.45.2

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

Posted by Michael Tokarev 1 year, 4 months ago

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

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

Posted by Richard Henderson 1 year, 5 months ago

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~