This is the mail archive of the gdb-patches@sources.redhat.com 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]

Re: [rfc/mi] ui_list_* to ui_*


Nice.

My only concerns are with code documentation (see below).  

P.S.: Eli mentioned the manual.  I remember that we wrote a document
describing the ui_out calls and I thought it was texi-ized already...


Andrew Cagney wrote:
> 
>         (struct ui_out): Replace ``list_flag'' and ``field_count'' with
>         ``level'' and ``levels''.

These names "thing" and "things" are sort of confusing.  Isn't there a
better way to describe these?


> + /* Level categories. */
> + enum ui_out_type
> +   {
> +     UI_OUT_TUPPLE,
> +     UI_OUT_LIST
> +   };
> +

I guess some comments defining what is a UI_OUT_TUPPLE (and even a
UI_OUT_LIST) are in order here.


Some comments before the top/pop_level stuff and struct ui_out_level
would not hurt as well.  It is always good to describe these objects. 
Think of it as a class description. 



 

-- 
Fernando Nasser
Red Hat Canada Ltd.                     E-Mail:  fnasser@redhat.com
2323 Yonge Street, Suite #300
Toronto, Ontario   M4P 2C9


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