[PATCH 2/2] mi-out: Implement mi redirection using a stack.

Tom Tromey tromey@redhat.com
Thu Jul 24 20:34:00 GMT 2014


>>>>> "Adrian" == Adrian Sendroiu <adrian.sendroiu@freescale.com> writes:

Adrian> gdb/
Adrian> 2014-07-23  Adrian Sendroiu  <adrian.sendroiu@freescale.com>

Adrian> 	* mi/mi-out.c (ui_filep): New typedef.
[...]

I think it would be good to have some text in the message describing the
rationale for this patch.

One nit below.

Adrian>    /* FIXME: This code should be using a ``string_file'' and not the
Adrian>       TUI buffer hack. */

Not your problem but I wonder what this FIXME comment means.
I suspect it's obsolete since as far as I know mem_fileopen does return
a "string_file".

Adrian> +  new_stream = mem_fileopen();

Space before the open paren.

The patch is ok with this fixed and with some rationale text.

thanks,
Tom



More information about the Gdb-patches mailing list