GNU bug report logs - #30077
27.0.50; print-quoted should default to t

Previous Next

Package: emacs;

Reported by: Stefan Monnier <monnier <at> IRO.UMontreal.CA>

Date: Thu, 11 Jan 2018 16:32:01 UTC

Severity: wishlist

Found in version 27.0.50

Done: Stefan Monnier <monnier <at> IRO.UMontreal.CA>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 30077 in the body.
You can then email your comments to 30077 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#30077; Package emacs. (Thu, 11 Jan 2018 16:32:01 GMT) Full text and rfc822 format available.

Acknowledgement sent to Stefan Monnier <monnier <at> IRO.UMontreal.CA>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Thu, 11 Jan 2018 16:32:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
To: bug-gnu-emacs <at> gnu.org
Subject: 27.0.50; print-quoted should default to t
Date: Thu, 11 Jan 2018 11:31:09 -0500
Package: Emacs
Version: 27.0.50


I was reviewing some of my local patches and noticed that I tend to
let-bind print-quoted to t everywhere, and it occurred to me that
instead we should simply change the default.  I can't remember
a circumstance where I needed print-quoted to be nil (tho I wouldn't be
surprised if this patch ends up finding such a circumstance).

I suggest the patch below (for master, obviously).


        Stefan


diff --git a/src/print.c b/src/print.c
index 2c5cf108aa..b9d556c17c 100644
--- a/src/print.c
+++ b/src/print.c
@@ -2424,7 +2424,7 @@ This affects only `prin1'.  */);
   DEFVAR_BOOL ("print-quoted", print_quoted,
 	       doc: /* Non-nil means print quoted forms with reader syntax.
 I.e., (quote foo) prints as \\='foo, (function foo) as #\\='foo.  */);
-  print_quoted = 0;
+  print_quoted = true;
 
   DEFVAR_LISP ("print-gensym", Vprint_gensym,
 	       doc: /* Non-nil means print uninterned symbols so they will read as uninterned.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#30077; Package emacs. (Thu, 11 Jan 2018 16:46:01 GMT) Full text and rfc822 format available.

Message #8 received at 30077 <at> debbugs.gnu.org (full text, mbox):

From: Robert Pluim <rpluim <at> gmail.com>
To: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
Cc: 30077 <at> debbugs.gnu.org
Subject: Re: bug#30077: 27.0.50; print-quoted should default to t
Date: Thu, 11 Jan 2018 17:44:59 +0100
Stefan Monnier <monnier <at> IRO.UMontreal.CA> writes:

> Package: Emacs
> Version: 27.0.50
>
>
> I was reviewing some of my local patches and noticed that I tend to
> let-bind print-quoted to t everywhere, and it occurred to me that
> instead we should simply change the default.  I can't remember
> a circumstance where I needed print-quoted to be nil (tho I wouldn't be
> surprised if this patch ends up finding such a circumstance).
>
> I suggest the patch below (for master, obviously).

Aagh, my customize settings changed!

But that doesn't break anything as far as I can see.

+1 from me

Robert




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#30077; Package emacs. (Thu, 11 Jan 2018 17:54:01 GMT) Full text and rfc822 format available.

Message #11 received at 30077 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
Cc: 30077 <at> debbugs.gnu.org
Subject: Re: bug#30077: 27.0.50; print-quoted should default to t
Date: Thu, 11 Jan 2018 19:53:21 +0200
> From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
> Date: Thu, 11 Jan 2018 11:31:09 -0500
> 
> I suggest the patch below (for master, obviously).
> 
> 
>         Stefan
> 
> 
> diff --git a/src/print.c b/src/print.c
> index 2c5cf108aa..b9d556c17c 100644
> --- a/src/print.c
> +++ b/src/print.c
> @@ -2424,7 +2424,7 @@ This affects only `prin1'.  */);
>    DEFVAR_BOOL ("print-quoted", print_quoted,
>  	       doc: /* Non-nil means print quoted forms with reader syntax.
>  I.e., (quote foo) prints as \\='foo, (function foo) as #\\='foo.  */);
> -  print_quoted = 0;
> +  print_quoted = true;
>  
>    DEFVAR_LISP ("print-gensym", Vprint_gensym,
>  	       doc: /* Non-nil means print uninterned symbols so they will read as uninterned.

I have no opinion about the change, but if this is installed, please
mention the change in NEWS.

Thanks.




Reply sent to Stefan Monnier <monnier <at> IRO.UMontreal.CA>:
You have taken responsibility. (Mon, 15 Jan 2018 18:44:02 GMT) Full text and rfc822 format available.

Notification sent to Stefan Monnier <monnier <at> IRO.UMontreal.CA>:
bug acknowledged by developer. (Mon, 15 Jan 2018 18:44:02 GMT) Full text and rfc822 format available.

Message #16 received at 30077-done <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 30077-done <at> debbugs.gnu.org
Subject: Re: bug#30077: 27.0.50; print-quoted should default to t
Date: Mon, 15 Jan 2018 13:43:25 -0500
> I have no opinion about the change, but if this is installed, please
> mention the change in NEWS.

Done,


        Stefan




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Tue, 13 Feb 2018 12:24:05 GMT) Full text and rfc822 format available.

This bug report was last modified 7 years and 122 days ago.

Previous Next


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