id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc	field_blocks	field_dependson	field_seealso
798	Definitions in abi/include/abi/elf.h are non-conformant	Jiri Svoboda		"The definitions in abi/include/abi/elf.h are meant to correspond to definitions as specified by SysV ABI. This commit:

{{{
commit ca0e838006b1e44ee4570c6851ffbd7a5a132165
Author: Jiří Zárevúcky <zarevucky.jiri@gmail.com>
Date:   Mon Feb 11 16:28:25 2019 +0100

    Convert preprocessor macros in abi/ to C constructs
    
    Preprocessor macros are an obsolete concept and they complicate things.
    They are also completely unnecessary in most circumstances.
    
    This commit changes untyped numeric constants into anonymous enums,
    typed constants into static const variables, and function-like macros
    into functions.
}}}

changed macros to enums (which is less harmful) and introduced enum names (also not very harmful) and changed the case of ELF_ST_BIND, ELF_ST_TYPE, ELF_ST_INFO to lowercase (harmful).

Our elf-related headers should be SysV compatible to prevent confusion and compatibility problems. I suggest keeping them as close as to the spec as possible (keep constants as symbolic constants, keep macros as macros), keep identifiers as they should be."	defect	assigned	major		helenos/unspecified	mainline