GNU bug report logs - #39805
describe-variable fails to print some circular list values

Previous Next

Package: emacs;

Reported by: No Wayman <iarchivedmywholelife <at> gmail.com>

Date: Wed, 26 Feb 2020 21:50:01 UTC

Severity: normal

Tags: confirmed, fixed, patch

Found in versions 28.0.50, 26.1

Fixed in version 27.1

Done: Noam Postavsky <npostavs <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: No Wayman <iarchivedmywholelife <at> gmail.com>
To: Noam Postavsky <npostavs <at> gmail.com>
Cc: 39805 <at> debbugs.gnu.org, No Wayman <iarchivedmywholelife <at> gmail.com>
Subject: bug#39805: 28.0.50; scan-sexps (scan_lists) incorrectly parsing circular list
Date: Thu, 27 Feb 2020 23:01:41 -0500
> Mostly yes, but IMO it's kind of over-engineered.  I would do 
> just:
>
>     (defun workaround/describe-variable-print-bug 
>     (original-function &rest args)
>       "Put temp buffer in emacs-lisp-mode before `pp-buffer'.
>     https://debbugs.gnu.org/39805#8"
>       (advice-add #'pp-buffer :before #'emacs-lisp-mode)
>       (unwind-protect
>         (apply original-function args)
>         (advice-remove #'pp-buffer #'emacs-lisp-mode)))
>
>     (when (= emacs-major-version 26) ;; Work around Bug#39805.
>       (advice-add #'describe-variable :around
>               #'workaround/describe-variable-print-bug))

I assumed mine was doing a more than it needed to.
Thanks again for the patch and the workaround.




This bug report was last modified 5 years and 83 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.