[diffoscope] [PATCH] Implement ElfFile as a container of sections

Jérémy Bobbio lunar at debian.org
Mon Jan 18 19:03:07 CET 2016


Dhole:
> This works perfectly :) I've implemented it in the revision I'm attaching.
> 
> Let me know what you think.

Commited. Many thanks! This fixes Debian #808197.

I just made a tiny change:

> +class ElfContainer(Container):
> +    _sec_types = {'X': ElfCodeSection, 'S': ElfStringSection, '_': ElfSection}
       ^^^^^^^^^^

This is a constant, so I've renamed this to `SECTION_TYPES`.

> +                type = [self._sec_types[type] for type in flag if \
> +                        type in self._sec_types][0]

Even if it works, I think using `self` here is misleading as we are
using a class constant. I've replaced it with `ElfContainer`.

-- 
Lunar                                .''`. 
lunar at debian.org                    : :Ⓐ  :  # apt-get install anarchism
                                    `. `'` 
                                      `-   
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.reproducible-builds.org/pipermail/diffoscope/attachments/20160118/3d2074b8/attachment.sig>


More information about the diffoscope mailing list