Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Vladimir Prus <ghost@cs.msu.su>
To: gdb-patches@sources.redhat.com
Subject: Re: Add -var-info-path-expression command
Date: Tue, 04 Apr 2006 07:25:00 -0000	[thread overview]
Message-ID: <e0t6qq$700$2@sea.gmane.org> (raw)
In-Reply-To: <dvehar$icf$1@sea.gmane.org>


Any comments on this yet? I believe we've discussed the rationale for
-var-info-path-expression command in the main list, and agreed that it's
necessary.

- Volodya

> Hello,
> the below patch ports the -var-info-path-expression MI command from Apple
> version. It allows on get expression corresponding to any variable object,
> in language-specific syntax, which can be used, for example, to set
> watchpoint  on that expression.
> 
> Top-level changelog entry:
> 
> 2006-03-17  Vladimir Prus <ghost@cs.msu.su>,
>    port of changes on Apple branch:
>    2002-03-28  James Ingham <jingham@apple.com>
> 
>         * varobj.c
>         (child_exists): Take child index, not child name, for performance.
>         (get_type_deref): Pass out whether the original was a pointer or
>          not.
>         (path_expr_of_variable): New function.  This returns the full path
>         expression to a variable.  The intent here is that you could use
>         this to make a new root varobj corresponding to the child varobj
>         whose path expression you are getting.
>         (varobj_get_path_expr): New function.  External wrapper for
>         path_expr_of_variable.
>         (c_path_expr_of_child, cplus_path_expr_of_child,
>         java_path_expr_of_child): New functions, return the path expr of a
>         child in its parent.  Also caches the expr for later use.
>         (is_root_p): New convenience function, returns whether a variable
>         is a root. Then I changed all the uses of var->name to use
>         name_of_variable, and the test
>         for is this a root were changed to use is_root_p.
> 
>         * varobj.h: (varobj_get_path_expr): New function definition.
> 
> 
> MI changelog entry:
> 
> 2006-03-17  Vladimir Prus <ghost@cs.msu.su>,
>    port of changes on Apple branch:
>    2002-03-28  James Ingham <jingham@apple.com>
> 
> 
>         * mi-cmds.h: Added def'n of mi_cmd_var_info_path_expression.
>         * mi-cmds.c: Added var-info-path-expression to command list.
>         * mi-cmd-var.c (mi_cmd_var_info_path_expression) New function,
>         the MI interface to varobj_get_path_expression.



  reply	other threads:[~2006-04-04  7:25 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-03-17 18:37 Vladimir Prus
2006-04-04  7:25 ` Vladimir Prus [this message]
2006-04-04 13:26   ` Daniel Jacobowitz
2006-04-04 13:45     ` Vladimir Prus
2006-04-04 16:13       ` Daniel Jacobowitz
2006-04-04 18:44       ` Eli Zaretskii

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='e0t6qq$700$2@sea.gmane.org' \
    --to=ghost@cs.msu.su \
    --cc=gdb-patches@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox