Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: xgsa <xgsa@yandex.ru>
To: Tom Tromey <tromey@redhat.com>
Cc: gdb-patches@sourceware.org
Subject: Re: set print object on should affect MI varobjs (PR mi/13393)
Date: Thu, 23 Feb 2012 04:58:00 -0000	[thread overview]
Message-ID: <4F45BF8B.70009@yandex.ru> (raw)
In-Reply-To: <4F412637.2010405@yandex.ru>

Ping. Can anybody look at this patch?

BTW, the first part has already been checked in, so this patch could be 
applied on HEAD.

And one more question connected to these changes. Should I update 
documentation somehow (e.g. add some notes to the "show/set print 
object" commands description or to MI interface chapter)?

Thanks,
Anton

-------- Original message --------
> As the first patch was approved (not checked in though cause my 
> account hasn't been created yet) here is the second part of the 
> changes. Note, that this patch relies on changes from the previous one.
>
>
> Thanks,
> Anton
>
>
> gdb/testsuite/ChangeLog:
>
> 2011-12-02  Anton Gorenkov <xgsa@yandex.ru>
>
>         PR mi/13393
>
>         * gdb.mi/mi-var-rtti.cc: New file.
>         * gdb.mi/mi-var-rtti.exp: New file.
>
> gdb/ChangeLog:
>
> 2011-12-02  Anton Gorenkov <xgsa@yandex.ru>
>
>         PR mi/13393
>
>         * value.c (value_actual_type): New function.
>         * value.h (value_actual_type): New declaration.
>         * varobj.c (varobj_create): Call value_actual_type instead of
>         value_type.
>         (create_child_with_value): Call value_actual_type instead of
>         value_type.
>         (value_of_root): Support for type change if the value changed 
> and RTTI
>         is used to determine the type.
>         (adjust_value_for_child_access): Extended with a new parameter 
> and cast
>         given value to enclosing type if it is set.
>         (c_number_of_children): Updated for extended
>         adjust_value_for_child_access.
>         (c_describe_child): Updated for extended
>         adjust_value_for_child_access.
>         (cplus_number_of_children): Updated for extended
>         adjust_value_for_child_access.
>         (cplus_describe_child): Updated for extended
>         adjust_value_for_child_access.
>
>



  reply	other threads:[~2012-02-23  4:24 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-02 19:50 set print object on should affect MI varobjs (PR 13393) xgsa
2011-12-08  8:15 ` xgsa
2011-12-19 20:34   ` xgsa
2011-12-20 17:47 ` Tom Tromey
2011-12-21 19:01   ` set print object on should affect MI varobjs (PR gdb/13393) xgsa
2011-12-21 19:37     ` Jan Kratochvil
2011-12-22 10:17       ` set print object on should affect MI varobjs (PR mi/13393) xgsa
2011-12-24  1:55         ` Jan Kratochvil
2011-12-24 13:56           ` xgsa
2012-01-02  2:22             ` Jan Kratochvil
2012-01-06 15:47               ` RTTI type improvement for (was: "Re: set print object on should affect MI varobjs (PR mi/13393)") xgsa
2012-01-09 14:41                 ` Jan Kratochvil
2012-01-11 21:58                   ` RTTI type improvement for Tom Tromey
2012-01-12 11:25                     ` xgsa
2012-02-06 21:45                 ` Tom Tromey
2012-02-08 18:34                   ` xgsa
2012-02-10 20:13                     ` Tom Tromey
2012-02-19 18:46                       ` set print object on should affect MI varobjs (PR mi/13393) xgsa
2012-02-23  4:58                         ` xgsa [this message]
2012-03-18 16:28                         ` xgsa
2012-03-18 20:41                           ` Eli Zaretskii
2012-03-19  7:10                             ` xgsa
2012-03-19 17:41                               ` Eli Zaretskii
2012-03-23 17:09                                 ` xgsa
2012-03-26 19:08                                   ` xgsa
2012-03-30 17:51                                   ` Tom Tromey
2012-03-30 18:01                                     ` Eli Zaretskii
2012-03-30 20:11                                     ` xgsa
2012-03-30 18:01                                   ` Eli Zaretskii
2012-03-30 20:25                                     ` xgsa
2012-03-30 20:52                                       ` Eli Zaretskii
2012-03-30 21:26                                         ` xgsa
2012-03-31  5:54                                           ` Eli Zaretskii
2012-03-31  6:57                                             ` xgsa
2012-03-31  9:33                                               ` xgsa
2012-04-03  0:54                                                 ` Doug Evans
2012-04-03 13:27                                                   ` xgsa
2012-04-06 17:11                                                     ` xgsa
2012-04-13  8:07                                                       ` xgsa
2012-04-13 12:22                                                         ` Eli Zaretskii
2012-04-13 12:34                                                           ` xgsa
2012-04-13 17:23                                                             ` Tom Tromey
2012-04-14 23:35                                                               ` xgsa
2012-04-16 18:35                                                                 ` Jan Kratochvil
2012-07-23 17:21                                                                   ` Ulrich Weigand
2012-08-06  7:26                                                                     ` xgsa
2012-02-21 14:15                       ` RTTI type improvement for xgsa

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=4F45BF8B.70009@yandex.ru \
    --to=xgsa@yandex.ru \
    --cc=gdb-patches@sourceware.org \
    --cc=tromey@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