public inbox for insight@sourceware.org
 help / color / mirror / Atom feed
* upcoming changes in GDB parsing routines
@ 2007-11-02 16:03 Joel Brobecker
  0 siblings, 0 replies; only message in thread
From: Joel Brobecker @ 2007-11-02 16:03 UTC (permalink / raw)
  To: insight

Hello Insight developers,

I am working on adding a new field in the various parsing routines
inside GDB. Currently, these parsing routines use the global
current_language to determine which language to use in order
to do the parsing, but now I want the language to be specifically
specified.

Some of the routines that are about to be changed are being used
by insight, and once I commit the patch, insight will no longer build
because of the missing parameter.  The simple fix is to add
"current_language" as an extra argument at the end of the argument list
for all the calls that break. This should give you the exact same
behavior as before. You might want to take this opportunity to review
the use of current_language to see if it is appropriate or not. I
suspect it will. In GDB, we have situations where we are re-evaluating a
breakpoint expression that might have been set using a language that's
different from the current_language.

I can try to fix insight for you, althought I don't know if I have
permissions to, and I'm not sure whether I'll be able to build it
or not. I'll send a message to this mailing-list as soon as the
patch is committed.

Cheers,
-- 
Joel

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2007-11-02 16:03 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-11-02 16:03 upcoming changes in GDB parsing routines Joel Brobecker

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).