From: Nick Roberts <nickrob@snap.net.nz>
To: Vladimir Prus <ghost@cs.msu.su>
Cc: gdb@sources.redhat.com
Subject: Re: -var-show-attributes response syntax
Date: Fri, 10 Nov 2006 20:47:00 -0000 [thread overview]
Message-ID: <17748.58492.703620.633216@kahikatea.snap.net.nz> (raw)
In-Reply-To: <ej2anl$9n8$1@sea.gmane.org>
> > I see no advantage in restricting the output but I've not used this
> > command. How do you want to use it?
>
> I want to add a new attribute there, actually, and I'd prefer to use more
> regular name=value syntax.
That's still a bit vague.
You say:
How about changing the above to "editable=0/1"?
Are you suggesting
^done,attr="editable=0", ^done,attr="editable=1"
or
^done,editable="0", ^done,editable="1"
Either case still requires the front end to do some string
manipulation/comparison. Can you state precisely how you would change the
format and precisely what the benefit would be?
> > > This sounds like
> > > breaking
> > > backward compatibility, but probably is not, because "editable" is
> > > broken itself:
> > >
> > > -var-create C * 1+1
> > > ^done,name="C",numchild="0",type="long"
> > > (gdb)
> > > -var-show-attributes C
> > > ^done,attr="editable"
> > > (gdb)
> >
> > Why do you think this is broken?
>
> Because you can't assign the value to "1+1" -- it's not lvalue. And trying
> to do so will result in error from gdb.
I see, I missed that. In fact it my example was also wrong too, constants
in C appear to be editable. Noneditables appear to be arrays, structures,
unions etc. However I think this a separate issue to changing the syntax.
--
Nick http://www.inet.net.nz/~nickrob
next prev parent reply other threads:[~2006-11-10 20:47 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-11-10 13:55 Vladimir Prus
2006-11-10 16:34 ` Nick Roberts
2006-11-10 16:53 ` Vladimir Prus
2006-11-10 20:47 ` Nick Roberts [this message]
2006-11-10 21:06 ` Vladimir Prus
2006-11-10 21:23 ` Nick Roberts
2006-11-10 17:02 ` Andreas Schwab
2006-11-10 21:46 ` Eli Zaretskii
2006-11-10 21:54 ` Nick Roberts
2006-11-10 23:10 ` Eli Zaretskii
2006-11-10 23:33 ` Nick Roberts
2006-11-11 9:26 ` Eli Zaretskii
2006-11-11 10:03 ` Nick Roberts
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=17748.58492.703620.633216@kahikatea.snap.net.nz \
--to=nickrob@snap.net.nz \
--cc=gdb@sources.redhat.com \
--cc=ghost@cs.msu.su \
/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