On Mon, 21 Oct 2019 at 14:54, Laurent Vivier <[email protected]> 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". thanks -- PMM
