public inbox for gdb-prs@sourceware.org
help / color / mirror / Atom feed
* [Bug gdb/16455] New: "set debug target" should not rewrite current_target
@ 2014-01-15 16:53 tromey at redhat dot com
  0 siblings, 0 replies; only message in thread
From: tromey at redhat dot com @ 2014-01-15 16:53 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=16455

            Bug ID: 16455
           Summary: "set debug target" should not rewrite current_target
           Product: gdb
           Version: HEAD
            Status: NEW
          Severity: normal
          Priority: P2
         Component: gdb
          Assignee: unassigned at sourceware dot org
          Reporter: tromey at redhat dot com

Currently, "set debug target" is implemented by rewriting
current_target.  This forces some of the target-squashing
inheritance to remain in place -- at least for to_shortname,
but perhaps other things as well.

It would be better if "set debug target" were implemented
as ordinary code in the various target API functions.  That
is, a macro like:

#define target_post_attach(pid) \
     (*current_target.to_post_attach) (&current_target, pid)

should be changed into an ordinary function definition
that performs the method call and then checks "targetdebug".
Many target functions already work this way, but not all.

See also bug #8688.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


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

only message in thread, other threads:[~2014-01-15 16:53 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-01-15 16:53 [Bug gdb/16455] New: "set debug target" should not rewrite current_target tromey at redhat dot com

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).