Re: [PATCH] icomplete-vertical

[Juri Linkov]

> Okay.  Here is the final (?) version of the patch; I forgot to update the
> docs in the previous one.

I tried it out, and everything works nicely.
The only problem I see this code is too hackish:

> +(defun icomplete--vertical-reformat-completions ()
> +  "Reformat the completion candidates returned by `icomplete-completions'."
> +  (save-match-data
> +    (setq icomplete-current-completions
> +          (if (string-match "^\\((.*)\\|\\[.*\\]\\)?{\\(\\(?:.\\|\n\\)+\\)}"
> +                            icomplete-current-completions)
> +              (format "%s \n%s"
> +                      (or (match-string 1 icomplete-current-completions) "")
> +                      (match-string 2 icomplete-current-completions))
> +            icomplete-current-completions))))

I guess like there are variables that affect formatting of the output
e.g. 'icomplete-hide-common-prefix', there could be a new similar
variable 'icomplete-add-initial-newline' to add a newline in
the 'icomplete-completions' formatting function.