bug#36324: 27.0.50; Error in savehist-save: (wrong-type-argument listp t)

[Michael Heerdegen]

Davor Rotim <rotim.davor@gmail.com> writes:

>   read-string("Confirm password: " nil t nil)
>   read-passwd("Confirm password: " nil nil)

Ah - AFAIU this binds `minibuffer-history-variable' to t in the
minibuffer.

So this part of Lars' commit:

-  (unless (or (eq minibuffer-history-variable t)
-             ;; XEmacs sets minibuffer-history-variable to t to mean "no
-             ;; history is being recorded".
-             (memq minibuffer-history-variable savehist-ignored-variables))
+  (unless (memq minibuffer-history-variable savehist-ignored-variables)

is the culprit.  We should only update the comment since Gnu Emacs is
doing this as well.

Lars, if you agree, could you please do this?

Thanks,

Michael.