This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH 06/10] Java language
On Monday, June 04 2012, Tom Tromey wrote:
>>>>>> "Sergio" == Sergio Durigan Junior <sergiodj@redhat.com> writes:
>
> Sergio> -static void insert_exp (int, struct expression *);
> Sergio> +static void insert_exp (struct parser_state *, int, struct expression *);
>
> If you ever want to do another cleanup in this area, here's an
> opportunity :)
Great!
> IMNSHO language parsers should not be manipulating the expout data
> structure like this. copy_exp is likewise naughty.
Hm, do you have some idea here? Some kind of abstraction, like putting
these functions on gdb/{parser-defs.h,parse.c}, or something better?
Thanks,
--
Sergio