This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: PATCH/c++testsuite: Fix a missing (void|)
- From: David Carlton <carlton at math dot stanford dot edu>
- To: Daniel Jacobowitz <drow at mvista dot com>
- Cc: gdb-patches at sources dot redhat dot com
- Date: 14 Jan 2003 09:08:12 -0800
- Subject: Re: PATCH/c++testsuite: Fix a missing (void|)
- References: <20030114044342.GA3587@nevyn.them.org>
On Mon, 13 Jan 2003 23:43:42 -0500, Daniel Jacobowitz <drow@mvista.com> said:
> When David added printmethod.exp, I think he didn't have a GCC 2.95 setup to
> test with. Outdated as it may be, I've still got one (not for much
> longer...). So here's a patch to accept the GCC 2.95 output, which is also
> correct.
> Previously accepted:
> print theA->virt
> $1 = &A::virt()
> (gdb) PASS: gdb.c++/printmethod.exp: print virtual method.
> Newly also accepted:
> print theA->virt
> $1 = &A::virt(void)
> (gdb) PASS: gdb.c++/printmethod.exp: print virtual method.
> This matches our style in the rest of the testsuite.
Fair enough. I don't think that void should really be there in an
ideal world, but it's certainly not worth worrying about.
And yes, I do have a 2.95 setup, I'm just not as good as I could be
about testing with it. (It's not my default compiler: the C++ code
that I write doesn't work with 2.95.) Right now I'm trickling in
fixes for 3.1/DWARF-2 output; once that's done, I should go back and
look at 2.95.
David Carlton
carlton@math.stanford.edu