[patch 3/4] Provide elf_scnfindvma

Mark Wielaard mjw at redhat.com
Wed Nov 13 12:03:21 UTC 2013


On Wed, 2013-11-06 at 22:10 +0100, Jan Kratochvil wrote:
>     Provide elf_scnfindvma
>     
>     libdwfl/
>     2013-11-06  Jan Kratochvil  <jan.kratochvil at redhat.com>
>     
>     	Provide elf_scnfindvma.
>     	* dwfl_module_addrsym.c (dwfl_module_addrsym) (same_section): Call
>     	elf_scnfindvma where is the code from here moved out.
>     
>     libelf/
>     2013-11-06  Jan Kratochvil  <jan.kratochvil at redhat.com>
>     
>     	Provide elf_scnfindvma.
>     	* Makefile.am (libelf_a_SOURCES): Add elf_scnfindvma.c.
>     	* elf_scnfindvma.c: New file.
>     	* libelf.h (elf_scnfindvma): New declaration.
>     	* libelf.map (ELFUTILS_1.156): New with elf_scnfindvma.

Although the code itself looks fine it should not be added to libelf.
Please just make this an internal libdwfl function if you want to reuse
it. Or just duplicate the code if not easily shared, it is small enough.

There are other implementation of libelf and gelf and we should try to
keep them somewhat compatible. There are some small differences already,
but lets not make them bigger.

Thanks,

Mark



More information about the elfutils-devel mailing list