Re: [patch] Fix uiout for execute_command_to_string

Tom Tromey
2010-09-10T20:27:37Z

>>>>> "Jan" == Jan Kratochvil <jan.kratochvil@redhat.com> writes:

Jan> Updated the patch with this part. Paul Bolle said he does not have
Jan> a signed FSF copyright assignment; if it matters in this case.

It doesn't, that patch is short enough.

Jan> 2010-09-09 Jan Kratochvil <jan.kratochvil@redhat.com>
Jan> Paul Bolle <pebolle@tiscali.nl>

One nit.

Jan> +struct cleanup *
Jan> +make_cleanup_ui_out_redirect_pop (struct ui_out *uiout)
Jan> +{
Jan> + return make_my_cleanup (&cleanup_chain, do_ui_out_redirect_pop, uiout);
Jan> +}

The two new functions need header comments.
Ok with that change.

Tom