Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@false.org>
To: Jim Blandy <jimb@red-bean.com>
Cc: Andrew STUBBS <andrew.stubbs@st.com>, gdb-patches@sources.redhat.com
Subject: Re: [RFC] Alternate approach to keeping convenience variables
Date: Sun, 22 Jan 2006 21:04:00 -0000	[thread overview]
Message-ID: <20060122210401.GG27224@nevyn.them.org> (raw)
In-Reply-To: <8f2776cb0601041037i3993da95te76e1d9daa86a23c@mail.gmail.com>

On Wed, Jan 04, 2006 at 10:37:06AM -0800, Jim Blandy wrote:
> On 1/4/06, Andrew STUBBS <andrew.stubbs@st.com> wrote:
> > Perhaps you could modify copy_type_recursive such that it creates a
> > cleanup chain specific to the internal variable. Then call do cleanups
> > in set_internal_var.
> >
> > Might be tricky with shared type copies though. Just a thought.
> 
> Exactly.
> 
> You can keep throwing complexity into the code to deal with this stuff
> until it buries you.  And it'll never be a full solution.  Or it'll be
> complete until someone wants to add some perfectly reasonable feature
> (just as is happening with "let's preserve the values of convenience
> variables across symfile loads" right now).  At some point, we have to
> take a step back and think about a complete solution.

Right.  For now, I'm OK with saying "this is a small memory leak when
symbol files are unloaded".  If someone wants to investigate adding
boehm-gc to GDB and using that, instead, then I'm OK with that too :-)

Every once in a while I do get the urge to go through GDB with a
leak checker.  But not often enough to jump through huge, rarely tested
hoops for that case.

-- 
Daniel Jacobowitz
CodeSourcery


  reply	other threads:[~2006-01-22 21:04 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-11-21 22:05 [PATCH] keeping convenience variables (take 2) Andrew STUBBS
2005-11-22  4:58 ` Eli Zaretskii
2005-11-22  8:43 ` Jim Blandy
2005-11-22 19:27   ` Andrew STUBBS
2005-12-10  4:46     ` [RFC] Alternate approach to keeping convenience variables Daniel Jacobowitz
2005-12-10  5:07       ` Jim Blandy
2005-12-10  8:24         ` Daniel Jacobowitz
2005-12-10 22:20           ` Jim Blandy
2005-12-11 18:12             ` Eli Zaretskii
2005-12-11 19:17       ` Eli Zaretskii
2006-01-23 22:29         ` Jim Blandy
2006-01-24 11:44           ` Andrew STUBBS
2006-01-24 18:41             ` Jim Blandy
2006-01-24 18:43               ` Daniel Jacobowitz
2006-01-24 19:16                 ` Jim Blandy
2006-01-24 19:24                   ` Daniel Jacobowitz
2006-01-25  0:13                     ` Jim Blandy
2006-01-24 19:45                   ` Andrew STUBBS
2006-01-04 12:17       ` Andrew STUBBS
2006-01-04 17:00         ` Jim Blandy
2006-01-04 17:48           ` Andrew STUBBS
2006-01-04 18:37             ` Jim Blandy
2006-01-22 21:04               ` Daniel Jacobowitz [this message]
2006-01-22 21:31       ` Daniel Jacobowitz
2006-01-23 22:46         ` Jim Blandy
2006-01-27 17:53         ` Eli Zaretskii
2006-01-27 18:12           ` Jim Blandy
2006-01-27 18:29             ` Eli Zaretskii
2006-01-27 18:41           ` Joel Brobecker
2006-02-01 23:14         ` Daniel Jacobowitz
2006-02-02  4:18           ` Eli Zaretskii
2006-02-06 22:14             ` Daniel Jacobowitz

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=20060122210401.GG27224@nevyn.them.org \
    --to=drow@false.org \
    --cc=andrew.stubbs@st.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=jimb@red-bean.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