Thanks for pointing this out. This is now fixed
and includes the name problem the header.

I am wanting to turn this on my default.

  https://gitlab.com/emacs-vm/vm/-/issues/515

Dr Rainer Woitok <[email protected]> writes:
> On Wednesday, 2025-09-17 15:59:48 -0400, John Stoffel wrote:
> 
> > ...
> > Can you maybe post the patches you have which aren't in the latest
> > release?  That might be something for all of us to enjoy using.
> 
> The only remaining patch which might be of general interest is:
> 
> diff --git a/lisp/vm-reply.el b/lisp/vm-reply.el
> --- a/lisp/vm-reply.el
> +++ b/lisp/vm-reply.el
> @@ -664,7 +664,7 @@
>       (if (re-search-forward "[^, \t][ \t]*\n[ \t\n]+[^ \t\n]"
>                              (vm-matched-header-contents-end)
>                              t)
> -         (error "tale is an idiot, and so are you. :-)"))
> +         (error "missing trailing comma in multi-line header"))
>       (goto-char (vm-matched-header-end))))))
>  
>  (defun vm-mail-mode-insert-message-id-maybe ()
> 
> I applied it because I always found this error message both,  derogative
> and absolutely vacuous.   However, the above patch to function "vm-help-
> tale" is of the sort  "least effort",  and it only works  because I have
> "vm-tale-is-an-idiot" enabled.
> 
> In my opinion error messages should never be suppressed BY DEFAULT,  and
> this error message in particular should not be suppressed at all.   Thus
> I would suggest replacing the form
> 
>   (if vm-tale-is-an-idiot
>       (vm-help-tale))
> 
> in function "vm-mail-send"  with the body of function "vm-help-tale" and
> then ditching both,  function "vm-help-tale"  and configuration variable
> "vm-tale-is-an-idiot".
> 
> Sincerely,
>   Rainer

Reply via email to