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]
Other format: [Raw text]

Re: [PATCH] mi-symbol-*


On Thu, Nov 07, 2002 at 11:59:52AM -0500, Elena Zannoni wrote about 'Re: [PATCH] mi-symbol-*':
> Info_line should return the core addr ranges for a given line number.
> You instead have implemented a different behavior: given a line number
> name, print the line number which we just gave as input, and the file name.
> Also in the usage and error messages, the use of the word 'symbol' is a bit
> inappropriate, because the argument of the command is not a symbol name.
Is there any documentation about the internals of gdb and what the
various terms mean? I do have some knowledge of how executables are
structured, but I don't know any details or specific terms.

>  > +enum mi_cmd_result
>  > +mi_cmd_symbol_info_function (char *command, char **argv, int argc)
>  > +{
>  > +  char *function = "";
>  > +  struct symtabs_and_lines sals;
>  > +  struct symbol *sym;
>  > +  struct minimal_symbol *msym;
>  > +  struct cleanup *old_chain;
>  > +  char *args, **canonical;
>  > +
>  > +  if (argc != 1)
>  > +    error ("mi_cmd_symbol_info_function: Usage: SYMBOL");
>  > +
>  > +  args = argv[0];
>  > +
>  > +  sals = decode_line_1 (&args, 1, NULL, 0, &canonical);
> 
> this one could be ok, but there is contradictory information in the
> current documentation. It was documented as taking a symbol and
> returning the function where the symbol is in, however it suggest that
> it should provide the functionality that a gdbtk function
> provides. That gdbtk function however takes a linespec as argument.
> 
> I think this mi command needs to be revisited.
> So, wait until the issue is cleared.
Any news on the interface?

Thanks,

Jelmer


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