Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Andrew Cagney <ac131313@cygnus.com>
To: Michael Snyder <msnyder@cygnus.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [RFA] Remote symbol look-up (resubmission)
Date: Mon, 14 May 2001 13:37:00 -0000	[thread overview]
Message-ID: <3B004215.5040502@cygnus.com> (raw)
In-Reply-To: <3B002163.D17BFA0E@cygnus.com>

> And here I assume you also mean "qSymbol".
> Therefore I don't understand your question.
> You seem to be suggesting that the documentation is different
> from what you think the code is doing?  But your two examples
> look to me to be identical except for what I assume are typos.


Hmm, checking the web site.  By the time it reached here the doco had 
``SymbSymbol''.  I think I'll blame my mailer :-(

Sorry, my bad.


>> My understanding of the most recent discussion was that the interaction
>> was going to be:
>> 
>> -> qSymbol
>> <- "" - unknown
>> "OK" - done
>> "qSymbol:<symbol>"
> 
> 
> I understood you to suggest that "qSymbol" was a more logical string
> than "qSharedObject", but I did not understand you to be saying that
> I should omit the object filename.  I'd like to keep it, against the
> possibility of future need.  There is a use that I could have made with
> it, I simply postponed doing so.  Others might have other uses for it, 
> especially if it included the full path.


See:
http://sources.redhat.com/ml/gdb-patches/2001-05/msg00154.html


>> and then
>> 
>> -> qSymbol:<value>:<symbol>
>> <- same return values
>> 
>> because the symbol file wasn't, in its self, useful to the target.  The
>> qSymbol without arguments indicated new symbols were available.
> 
> 
> The symbol file could be useful to the target.  For instance, we could
> specify to GDB which symbol file was to be used for the lookup.  This
> would be consistent with the usage in the original thread-db spec from Sun.


You've lost me here.

Are you saying that there is going to need to be an extra parameter (the 
shared library name) added to the target->gdb symbol request on Solaris? 
  That would make it:

	qSymbol:<symbol> [ : <library> ]


>> However, if you think the target should be notified of each new symbol
>> file then I'd rather see protocol go back to ``[qQ]SymbolFile:<file>''
>> followed by ``[qQ]Symbol:<val>:<sym>'' rather than the very subtlely
>> different ``QSymbol'' vs ``qSymbol''.
> 
> 
> I'll be glad to go back to that syntax.


If it can be justified.

Remember, it is really important to get these protocol changes right. 
Unlike gdb internals, this is an external interface and once defined is 
largely set in concrete.

	Andrew



  reply	other threads:[~2001-05-14 13:37 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-05-11 10:26 Michael Snyder
2001-05-11 12:39 ` Eli Zaretskii
2001-05-11 18:51   ` Michael Snyder
2001-05-11 19:57 ` Andrew Cagney
2001-05-14 11:18   ` Michael Snyder
2001-05-14 13:37     ` Andrew Cagney [this message]
2001-05-14 14:34       ` Michael Snyder
2001-05-17 20:44         ` Andrew Cagney
2001-05-21 15:12           ` Michael Snyder
2001-05-21 15:30           ` Michael Snyder
2001-06-06  9:07             ` Andrew Cagney

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=3B004215.5040502@cygnus.com \
    --to=ac131313@cygnus.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=msnyder@cygnus.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