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 v3 06/34] Introduce interpreter factories


On 05/18/2016 08:19 PM, Simon Marchi wrote:
> On 16-05-06 08:34 AM, Pedro Alves wrote:

>> diff --git a/gdb/interps.h b/gdb/interps.h
>> index f0badc5..3065fdf 100644
>> --- a/gdb/interps.h
>> +++ b/gdb/interps.h
>> @@ -24,6 +24,17 @@
>>  
>>  struct ui_out;
>>  struct interp;
>> +struct ui;
>> +
>> +typedef struct interp *(*interp_factory_func) (const char *interp,
> 
> Nit:
> 
> In the functions of this type ({cli,mi,tui}_interp_factory), this parameter is named
> "name", so you could as well name it "name" here as well.

I've done that now.

> 
>> +					       struct ui *ui);
> 
> Even in the final result (with all patches applied), none of the factories
> use the ui parameter, is it expected?

I think that at some point I thought of storing a pointer to the UI in
the interp, but then ended up not needing it.  I've removed that
parameter now.

I'll post a patch in response to your other email.

Thanks,
Pedro Alves


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