Re: [Emacs-diffs] master 39355bc: Revert "* lisp/subr.el (when): Use `macroexp-progn'"

[Ken Brown]

On 11/4/2015 9:22 AM, Artur Malabarba wrote:
> branch: master
> commit 39355bc045874874eb83cecdf2e2e04c3b81d3f7
> Author: Artur Malabarba <address@hidden>
> Commit: Artur Malabarba <address@hidden>
>
>      Revert "* lisp/subr.el (when): Use `macroexp-progn'"
>
>      This reverts commit 8e843831eaf271801836b7a3e4dd3b4fb0bb72b8.
>      It breaks bootstrapping (duh).
> ---
>   lisp/subr.el |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/lisp/subr.el b/lisp/subr.el
> index 91647a6..ea926ae 100644
> --- a/lisp/subr.el
> +++ b/lisp/subr.el
> @@ -179,7 +179,7 @@ value of last one, or nil if there are none.
>
>   \(fn COND BODY...)"
>     (declare (indent 1) (debug t))
> -  (list 'if cond (macroexp-progn body)))
> +  (list 'if cond (cons 'progn body)))
>
>   (defmacro unless (cond &rest body)
>     "If COND yields nil, do BODY, else return nil.

You only reverted part of the commit, leaving the new test in place. 
Was that intentional?

Ken