[PATCH 2/7] move platform depended include into system.h

罗勇刚(Yonggang Luo) luoyonggang@gmail.com
Sat Oct 15 17:01:29 GMT 2022


OK, updated in v2

On Sat, Oct 15, 2022 at 5:16 AM Mark Wielaard <mark@klomp.org> wrote:
>
> On Tue, Sep 20, 2022 at 04:43:02PM +0800, Yonggang Luo via Elfutils-devel
wrote:
> > All of these files either #include <system.h> directly or #include
"libelfP.h"
> > And now "libelfP.h also #include <system.h>, so the platform depended
include
> > can be moved to system.h safely
>
> I like this in theory since it cleans up some of the includes.
> But it doesn't work as is.
>
> libebl/eblobjnotetypename.c only included system.h so now doesn't
> compile anymore.  And libintl.h is removed from libelf/elf_error.c
> which really is necessary.
>
> Cheers,
>
> Mark



--
         此致
礼
罗勇刚
Yours
    sincerely,
Yonggang Luo


More information about the Elfutils-devel mailing list