From: Jan Kratochvil <jan.kratochvil@redhat.com>
To: Pedro Alves <pedro@codesourcery.com>
Cc: gdb-patches@sourceware.org, pebolle@tiscali.nl
Subject: Re: [patch] Fix nesting of ui_out_redirect
Date: Fri, 03 Sep 2010 18:01:00 -0000 [thread overview]
Message-ID: <20100903154218.GA5060@host1.dyn.jankratochvil.net> (raw)
In-Reply-To: <201009031633.23542.pedro@codesourcery.com>
On Fri, 03 Sep 2010 17:33:23 +0200, Pedro Alves wrote:
> On Friday 03 September 2010 16:23:32, Jan Kratochvil wrote:
>
> > gdb/
> > 2010-09-03 Jan Kratochvil <jan.kratochvil@redhat.com>
> >
> > * breakpoint.c (save_breakpoints): Use RETURN_MASK_ALL.
> > * cli-out.c: Include vec.h.
> > (cli_field_fmt, cli_spaces, cli_text, cli_message, cli_flush): New
> > variable stream, initialize it, use it.
> > (cli_redirect): New function comment. Replace the stream and
> > original_stream fields by the new streams field. Remove the
> > original_stream != NULL conditional, assert error on NULL instead.
> > (out_field_fmt, field_separator): New variable stream, initialize it, use it.
> > (cli_out_data_ctor): Assert non-NULL stream. Replace the stream and
> > original_stream fields by the new streams field.
> > (cli_out_set_stream): Replace the stream field by the new streams
> > field.
> > * cli-out.h: Include vec.h.
> > (ui_filep): New typedef, call DEF_VEC_P for it.
> > (struct cli_ui_out_data): Replace the stream and original_stream
> > fields by the new streams field.
> > * cli/cli-logging.c (set_logging_redirect): Call ui_out_redirect with
> > NULL first. Extend the comment.
> > (handle_redirections): Call ui_out_redirect with output.
> > * python/py-breakpoint.c (bppy_get_commands): Move ui_out_redirect
> > calls outside of the TRY_CATCH block.
> >
> > gdb/testsuite/
> > 2010-09-03 Jan Kratochvil <jan.kratochvil@redhat.com>
> >
> > * gdb.base/ui-redirect.exp: New file.
>
> Okay.
Checked-in:
http://sourceware.org/ml/gdb-cvs/2010-09/msg00031.html
Thanks,
Jan
next prev parent reply other threads:[~2010-09-03 15:42 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-03 15:33 Jan Kratochvil
2010-09-03 15:36 ` Pedro Alves
2010-09-03 15:40 ` Jan Kratochvil
2010-09-03 15:40 ` Pedro Alves
2010-09-03 18:01 ` Jan Kratochvil [this message]
2010-09-03 18:17 ` Pedro Alves
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=20100903154218.GA5060@host1.dyn.jankratochvil.net \
--to=jan.kratochvil@redhat.com \
--cc=gdb-patches@sourceware.org \
--cc=pebolle@tiscali.nl \
--cc=pedro@codesourcery.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).