This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [BUILDROBOT] error: redefinition of typedef ʽrecord_line_ftypeʼ (was: [PATCH 3/6] DWARF Two Level Line Tables: check_line_address)
- From: "Maciej W. Rozycki" <macro at linux-mips dot org>
- To: Doug Evans <dje at google dot com>
- Cc: Jan-Benedict Glaw <jbglaw at lug-owl dot de>, gdb-patches <gdb-patches at sourceware dot org>
- Date: Thu, 28 May 2015 18:27:22 +0100 (BST)
- Subject: Re: [BUILDROBOT] error: redefinition of typedef ʽrecord_line_ftypeʼ (was: [PATCH 3/6] DWARF Two Level Line Tables: check_line_address)
- Authentication-results: sourceware.org; auth=none
- References: <yjt261a67zlk dot fsf at ruffy dot mtv dot corp dot google dot com> <CADPb22RXqpCLCyvY8dw8bXEd+ArnThrBZfNitZWSAqgi4X=irg at mail dot gmail dot com> <20150528092738 dot GY7864 at lug-owl dot de> <alpine dot LFD dot 2 dot 11 dot 1505281448580 dot 21603 at eddie dot linux-mips dot org> <CADPb22SMqtn86MxgQP4E=f_EgcqA8n=8qpdEHyTheB5hqZAYZw at mail dot gmail dot com>
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