This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 03/10] C language


On Monday, June 04 2012, Doug Evans wrote:

> On Sat, Jun 2, 2012 at 12:32 PM, Sergio Durigan Junior
> <sergiodj@redhat.com> wrote:
>> @@ -2651,11 +2665,16 @@ yylex (void)
>> Â}
>>
>> Âint
>> -c_parse (void)
>> +c_parse (struct parser_state *par_state)
>> Â{
>> Â int result;
>> - Âstruct cleanup *back_to = make_cleanup (free_current_contents,
>> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â &expression_macro_scope);
>> + Âstruct cleanup *back_to;
>> +
>> + Â/* Setting up the parser state. Â*/
>> + Âgdb_assert (par_state != NULL);
>> + Âpstate = par_state;
>> +
>> + Âback_to = make_cleanup (free_current_contents, &expression_macro_scope);
>>
>> Â /* Set up the scope for macro expansion. Â*/
>> Â expression_macro_scope = NULL;
>
> IWBN to reset pstate back to NULL when done.

I did that in a first draft, but then removed it.  I will fix it for
every patch, then (but won't resubmit all of them because that would be
worthless).  Thanks.

-- 
Sergio


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]