Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Bob Rossi <bob@brasko.net>
To: Denis PILAT <denis.pilat@st.com>
Cc: Nick Roberts <nickrob@snap.net.nz>, gdb-patches@sources.redhat.com
Subject: Re: [PATCH] Target stderr not displayed thru MI
Date: Thu, 01 Dec 2005 14:06:00 -0000	[thread overview]
Message-ID: <20051201140436.GA31759@white> (raw)
In-Reply-To: <438F011F.7050403@st.com>

> I attach a new patch in that sense. It does not include the documentation 
> patch about MI new stream but it's just to give you an idea.
> Any comment ?

> +  /* MI 1 and 2 target error stream use the same steam prefix "@" in oder
> +     to ensure backward compatibility with old frondtend, MI 3 uses
> +     a new prefix "#" */
> +  if (mi_version (uiout) > 2)
> +    mi->targerr = mi_console_file_new (raw_stdout, "#", '"');
> +  else
> +    mi->targerr = mi_console_file_new (raw_stdout, "@", '"');
> +
>    mi->event_channel = mi_console_file_new (raw_stdout, "=", 0);

We have a lot of people that are confused about the target-stream-output
and it was recently brought up that maybe this wasn't used and should be
removed.

   http://sources.redhat.com/ml/gdb/2005-11/msg00391.html

However, you seem to be using it, and want to add another stream-output,
probably called something like target-stream-stderr-output.

I haven't seen MI use the target-stream-output yet. What is your
configuration that makes this happen? Is it still useful to you?
If so, it would be nice if we could improve the doco a bit, to let users
know when this would be useful for them.

Thanks,
Bob Rossi


  reply	other threads:[~2005-12-01 14:06 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-11-30 14:15 Nick Roberts
2005-11-30 18:57 ` [PATCH] " Denis PILAT
2005-11-30 19:03   ` Bob Rossi
2005-11-30 19:55     ` Daniel Jacobowitz
     [not found]   ` <17294.2894.345752.773239@kahikatea.snap.net.nz>
2005-12-01 13:57     ` Denis PILAT
2005-12-01 14:06       ` Bob Rossi [this message]
2005-12-01 14:32         ` Daniel Jacobowitz
2005-12-01 16:20           ` Denis PILAT
2005-12-01 22:49           ` Nick Roberts
2005-12-02  1:26             ` Jim Blandy
2005-12-05 20:50             ` Denis PILAT
2005-12-06  6:09               ` Nick Roberts
2005-12-06 10:49                 ` Daniel Jacobowitz
  -- strict thread matches above, loose matches on Subject: below --
2005-11-30  2:47 Denis PILAT
2005-11-30 13:51 ` Jim Blandy

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=20051201140436.GA31759@white \
    --to=bob@brasko.net \
    --cc=denis.pilat@st.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=nickrob@snap.net.nz \
    /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