[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 2/4] elf: move elf.h to elf/elf.h and split o
Re: [Qemu-devel] [PATCH v1 2/4] elf: move elf.h to elf/elf.h and split out types
Mon, 21 Oct 2019 15:04:11 +0100
On Mon, 21 Oct 2019 at 14:54, Laurent Vivier <address@hidden> wrote:
> Le 10/09/2019 à 21:34, Alex Bennée a écrit :
> > Most of the users of elf.h just want the standard Elf definitions. The
> > couple that want more than that want an expansion based on ELF_CLASS
> > which can be used for size agnostic code. The later is moved into
> > elf/elf-types.inc.h to make it clearer what it is for. While doing
> > that I also removed the whitespace damage.
> The patch looks good, but why did you call the file "elf-types.inc.h"
> and not "elf-types.h"?
This is our usual convention for files (well, that or .inc.c)
which get included multiple times with different preprocessor
defines to specify how they behave, i.e. which aren't standard
"simple header with multiple-inclusion protection guards".