public inbox for insight@sourceware.org
 help / color / mirror / Atom feed
From: Andrew Cagney <ac131313@ges.redhat.com>
To: Daniel Jacobowitz <drow@mvista.com>
Cc: gdb-patches@sources.redhat.com, insight@sources.redhat.com
Subject: Re: RFA: Make cli-out follow gdb_stdout
Date: Mon, 22 Jul 2002 13:11:00 -0000	[thread overview]
Message-ID: <3D3C66E3.50807@ges.redhat.com> (raw)
In-Reply-To: <20020722194126.GA8756@nevyn.them.org>


[...]

> Well, that's not helpful in the short term :P
> 
> What do you envision by explicitly parametrized i/o object?  I really
> don't think that's a good idea.  It would have to be passed down all
> the way to every target function which might want to print some kind of
> output.  That's a lot of bulk for no visible gain; somewhere down the
> road if we supported multiple instantiations of the gdb library, maybe,
> but we're so far away from that that this sort of direction seems
> futile, until we are at least passing an object cookie everywhere.

As they say, I don't yet know.  I think it is pretty clear that the I/O 
is just one part of this state problem.  Since of thread, frame, 
register cache, ... are heavily dependant on state implemented with 
globals a guess is an object (or object relationship) that contains that 
state.   In the mean time we need to ensure that we're not entrenching 
the problem (eg by adding another bit of code assuming global state).

For the immediate problem, the intent is for catch_exceptions() to be 
used when overriding gdb's I/O.  Given, looking at the code, it too 
fails to keep that assertion (oops):

 > 	global uiout->stream->ui_file == global gdb_stdout

I think the thing to do is fix catch-exceptions and then use that.

enjoy,
Andrew


  reply	other threads:[~2002-07-22 20:11 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-07-17 11:30 Daniel Jacobowitz
2002-07-22 11:12 ` Andrew Cagney
2002-07-22 11:21   ` Daniel Jacobowitz
2002-07-22 12:18     ` Andrew Cagney
2002-07-22 12:41       ` Daniel Jacobowitz
2002-07-22 13:11         ` Andrew Cagney [this message]
2002-07-22 13:18           ` 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=3D3C66E3.50807@ges.redhat.com \
    --to=ac131313@ges.redhat.com \
    --cc=drow@mvista.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=insight@sources.redhat.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).