This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch] Make test-cp-name-parser usable
- From: Daniel Jacobowitz <drow at false dot org>
- To: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Wed, 3 Aug 2011 16:36:49 -0400
- Subject: Re: [patch] Make test-cp-name-parser usable
- References: <20110802170152.GA24932@host1.jankratochvil.net>
On Tue, Aug 2, 2011 at 1:01 PM, Jan Kratochvil
<jan.kratochvil@redhat.com> wrote:
> gdb/
> 2011-08-02 ?Jan Kratochvil ?<jan.kratochvil@redhat.com>
>
> ? ? ? ?* cp-name-parser.y (xfree): Remove the call of free.
> ? ? ? ?(main): Uncomment "Demangling error\n".
OK.
> --- a/gdb/cp-name-parser.y
> +++ b/gdb/cp-name-parser.y
> @@ -2036,13 +2036,12 @@ trim_chars (char *lexptr, char **extra_chars)
> ?}
>
> ?/* When this file is built as a standalone program, xmalloc comes from
> - ? libiberty --- in which case we have to provide xfree ourselves. ?*/
> + ? libiberty --- in which case we have to provide xfree ourselves.
> + ? A call to free would get translated xfree back again. ?Leak is OK. ?*/
"translated back to xfree again"
What does the translation - is it as simple as #undef free or (free) (ptr)?
--
Thanks,
Daniel