This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH 4/4] compile: New 'compile print'
- From: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- To: Eli Zaretskii <eliz at gnu dot org>
- Cc: gdb-patches at sourceware dot org, pmuldoon at redhat dot com
- Date: Fri, 27 Mar 2015 08:33:33 +0100
- Subject: Re: [PATCH 4/4] compile: New 'compile print'
- Authentication-results: sourceware.org; auth=none
- References: <20150326205727 dot 28223 dot 54648 dot stgit at host1 dot jankratochvil dot net> <20150326205752 dot 28223 dot 24230 dot stgit at host1 dot jankratochvil dot net> <83iodmnc18 dot fsf at gnu dot org>
On Fri, 27 Mar 2015 08:18:43 +0100, Eli Zaretskii wrote:
> I suggest a different name for this command. Unfortunately, "eval" is
> already taken, but perhaps "parse" or "parse-eval"? Or maybe a new
> switch to "print"? "compile print" sounds awkward and unintuitive to
> me.
FYI there is already "compile code" and "compile file".
> > + add_cmd ("print", class_obscure, compile_print_command,
> > + _("\
> > +Evaluate a EXPR with the compiler and print result.\n\
>
> Suggest to drop the "a" part, it just makes this sentence harder to
> read.
OK, it will be changed upon check-in.
> > +The expression may be specified in one line, e.g.:\n\
>
> "on the same line as the command" sounds more clear to me.
OK, it will be changed upon check-in.
Thanks,
Jan