This is the mail archive of the
libstdc++@gcc.gnu.org
mailing list for the libstdc++ project.
Re: [PATCH] PR libstdc++/12736
- From: Carlo Wood <carlo at alinoe dot com>
- To: Benjamin Kosnik <bkoz at redhat dot com>
- Cc: libstdc++ at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
- Date: Fri, 7 Nov 2003 19:01:23 +0100
- Subject: Re: [PATCH] PR libstdc++/12736
- References: <Pine.LNX.4.44.0310171323570.898-100000@bellatrix.tat.physik.uni-tuebingen.de> <jeekxcnkza.fsf@sykes.suse.de> <20031017134223.GA11618@alinoe.com> <20031022113257.3fd660fb.bkoz@redhat.com> <20031022213252.GA14830@alinoe.com> <20031022164512.330a58fc.bkoz@redhat.com> <20031107032155.GA16549@alinoe.com> <20031107105012.3cc4cc28.bkoz@redhat.com>
On Fri, Nov 07, 2003 at 10:50:12AM -0600, Benjamin Kosnik wrote:
> > * testsuite/demangle/abi_text/01.cc: New.
>
> Carlo, this looks fine, but the "new" testcase already exists, and you
> are just modifying it. Just change the ChangeLog to reflect this and you're ok.
Oops, thanks. Actually I HAD added a new testsuite test
but a 'cvs diff' doesn't show it - that had confused me.
The real change log is:
* testsuite/demangle/abi_text/01.cc: Reordered CV-qualifiers.
* testsuite/demangle/regression/cw-16.cc: New.
and the new test is:
verify_demangle("_Z3fooIA6_KiEvA9_KT_rVPrS4_",
"void foo<int const [6]>(int const [9][6], int const restrict (* volatile restrict) [9][6])");
Committed anyway as obvious.
Thanks for your prompt reviews!
--
Carlo Wood <carlo@alinoe.com>