Re: [BUILDROBOT] error: redefinition of typedef ‘record_line_ftype’ (was: [PATCH 3/6] DWARF Two Level Line Tables: check_line_address)

Maciej W. Rozycki
2015-05-28T17:27:30Z

On Thu, 28 May 2015, Doug Evans wrote:

> > The typedef in buildsym.h has to stay, it's already used elsewhere.
> > It's the newly-added one in dwarf2read.c that has to go. Both are
> > identical anyway.
>
> Fixed.

Thanks!

> The build didn't fail for me so I didn't see this.
> I then tried several pure gcc versions (can't trust distro gcc defaults).
> I got a failure with 4.3.3 but not with 4.[6-9].x.

The version I've been using is 6.0 head, so I guess in the quest for
perfection GCC has become strictier since 4.9 here as well. It might be
worth trying 5.0, being a proper release rather than WIP.

Offhand I can't tell if the complaint was legitimate or not. I suspect
it was, even though the redefinition merely repeated the original, but in
any case the extra typedef was redundant.

Maciej