On 19.12.2022 15:45, Sergey Dyasli wrote:
> This allows to use them for forward declaration in other headers.
>
> Signed-off-by: Sergey Dyasli <sergey.dya...@citrix.com>
Acked-by: Jan Beulich <jbeul...@suse.com>
with ...
> --- a/xen/include/xen/multiboot.h
> +++ b/xen/include/xen/multiboot.h
> @@ -46,23 +46,25 @@
> #ifndef __ASSEMBLY__
>
> /* The symbol table for a.out. */
> -typedef struct {
> +struct aout_symbol_table {
> u32 tabsize;
> u32 strsize;
> u32 addr;
> u32 reserved;
> -} aout_symbol_table_t;
> +};
> +typedef struct aout_symbol_table aout_symbol_table_t;
>
> /* The section header table for ELF. */
> -typedef struct {
> +struct elf_section_header_table{
... the missing blank added here.
I also have to admit that I would have preferred a variant with less
code churn: The typedef re-arrangement really isn't needed to fulfill
the goal pf the change.
Jan
