bug#23092: 25.0.92; Minibuffer completion fails to resize completion window if reused during same command

[martin rudalics]

> However, after I looked at this again now, it looks natural to just
> replace ‘shrink-window-if-larger-than-buffer’ with ‘fit-window-to-buffer’.
> But I'd leave it to you to decide how good this change is and what
> consequences it might entail.

It should have two consequences: (1) When there are many completions,
the *Completions* window might be larger initially.  (2) The
*Completions* window will mostly behave as if ‘temp-buffer-resize-mode’
were enabled by default.  I think the OP's scenario is much more
embarrassing so I think we can live with the consequences of your fix.

Please install it and I'll fix the manual accordingly.

Thanks, martin

> diff --git a/lisp/minibuffer.el b/lisp/minibuffer.el
> index ecac0ae..6540059 100644
> --- a/lisp/minibuffer.el
> +++ b/lisp/minibuffer.el
> @@ -1835,7 +1835,7 @@ minibuffer-completion-help
>                   'display-buffer-below-selected))
>        ,(if temp-buffer-resize-mode
>             '(window-height . resize-temp-buffer-window)
> -         '(window-height . shrink-window-if-larger-than-buffer))
> +         '(window-height . fit-window-to-buffer))
>        ,(when temp-buffer-resize-mode
>           '(preserve-size . (nil . t))))
>             nil
>
>