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

Jan Kratochvil
2015-03-27T07:33:43Z

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