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

[Maciej W. Rozycki]

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