This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFC] delete gdb.cp/ambiguous.exp ?
- From: Pedro Alves <palves at redhat dot com>
- To: Doug Evans <dje at google dot com>
- Cc: gdb-patches <gdb-patches at sourceware dot org>
- Date: Tue, 19 Aug 2014 18:50:26 +0100
- Subject: Re: [RFC] delete gdb.cp/ambiguous.exp ?
- Authentication-results: sourceware.org; auth=none
- References: <yjt27g25tmac dot fsf at ruffy dot mtv dot corp dot google dot com> <53F38030 dot 1020406 at redhat dot com> <CADPb22TNGVOJ-WM9dQ5z_KetULps7ZfnXBG5DkxPOYTR00D8cw at mail dot gmail dot com>
On 08/19/2014 06:24 PM, Doug Evans wrote:
>> Actually enabling the test (removing the skip, and adding
>> nowarnings), we see that indeed GDB outputs no warning:
>
> But given the early exit the test itself is never run.
As I said, I removed that. ;-)
> And it's been this way since at least 2003 (commit 4d9eda44f), and
> longer (that commit just changed the style of the gcc test)!
Yeah, this probably came in in the big HP merge, much
earlier than that. There was once a gdb.hp/ambiguous.exp, even, and
this probably got copied from that.
There was once a big
everything-goes-we-dont-have-time-to-clean-things-up-before-accepting HP
import/contribution, and bits may be skipped on GCC just because the
HP folks at the time didn't want to deal with it in their local fork.
> I'm all for filing a bug and recording the test in the bug report.
> I'm even ok with keeping the test as is.
> The high order bit for me is exploring what the community wishes be
> done with this kind of "dead code".
You're asking for a generic opinion, of the abstract "community",
while I think "case by case" generally applies. ;-) My inclination for
tests is to first check whether there's something salvageable. If
someone wrote the test, it was probably important enough. If it's indeed
"dead code", then of I'd go for just removing it. I looked, and it seemed
to me that the test actually covers an aspect of printing that we don't
cover elsewhere, and actually reveals a bug.
So IMO, in this particular case, we should keep the test, remove the gcc check,
modernize and KFAIL it , and then have the bug fixed (if people agree it's
a bug). I'm of course not suggesting you do that all of yourself, but
since you asked, that's what I'd prefer see done in this case.
Thanks,
Pedro Alves