bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#64344: [PATCH] Make calculator work with customized mode-line


From: john muhl
Subject: bug#64344: [PATCH] Make calculator work with customized mode-line
Date: Thu, 29 Jun 2023 10:12:06 -0500

Eli Zaretskii <eliz@gnu.org> writes:

>> +               (and bx (or (not lh) (> (if (listp lh) (cdr lh) lh) 0))))
>                                                ^^^^^
> Shouldn't that be 'consp' instead?  'listp' returns non-nil for nil
> argument.

I’m not expert enough to say more than that either way works to fix the
issue. Originally I did the ‘(cdr lh)’ unconditionally and got an error
mentioning ‘listp’ so I just used that when I added the condition; if it
had said ‘conps’ I would have used that.

Here is the patch with ‘consp’ instead.

Attachment: 0001-lisp-calculator-el--calculator-need-3-lines---Ch.patch
Description: [PATCH] * lisp/calculator.el (calculator-need-3-lines): Check type


reply via email to

[Prev in Thread] Current Thread [Next in Thread]