This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 4/4] compile: New 'compile print'


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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]