bug#63861: closed ([PATCH] pp.el: New "pretty printing" code)

[GNU bug Tracking System]

Your message dated Sat, 17 Jun 2023 18:08:33 -0400
with message-id <jwvr0q92174.fsf-monnier+emacs@gnu.org>
and subject line Re: bug#63861: [PATCH] pp.el: New "pretty printing" code
has caused the debbugs.gnu.org bug report #63861,
regarding [PATCH] pp.el: New "pretty printing" code
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
63861: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=63861
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems

--- Begin Message --- Subject: [PATCH] pp.el: New "pretty printing" code Date: Fri, 02 Jun 2023 18:50:57 -0400

Tags: patch

I've often been annoyed by the way `ielm` "pretty prints" data,
so I wrote my own pretty printer, which has evolved over the years.
I believe it has finally reached a state which may be acceptable
to more than just myself.

The new code is in a new function `pp-region`.
The old code redirects to the new code if `pp-buffer-use-pp-region` is
non-nil, tho I'm not sure we want to bother users with such
a config var.  Hopefully, the new code should be good enough that users
don't need to choose.  Maybe I should make it a `defvar` and have it
default to t, so new users will complain if it's not good enough?


        Stefan


 In GNU Emacs 30.0.50 (build 2, x86_64-pc-linux-gnu, X toolkit, cairo
 version 1.16.0, Xaw3d scroll bars) of 2023-05-24 built on pastel
Repository revision: 41b6b907d4cf2f8c302647dc63c5d9c94f9f01d6
Repository branch: work
Windowing system distributor 'The X.Org Foundation', version 11.0.12011000
System Description: Debian GNU/Linux 11 (bullseye)

Configured using:
 'configure -C --enable-checking --enable-check-lisp-object-type --with-modules 
--with-cairo --with-tiff=ifavailable
 'CFLAGS=-Wall -g3 -Og -Wno-pointer-sign'
 PKG_CONFIG_PATH=/home/monnier/lib/pkgconfig'

0001-pp.el-New-pretty-printing-code.patch
Description: Text Data

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#63861: [PATCH] pp.el: New "pretty printing" code
>
>
>
>
> Date: 
>
> Sat, 17 Jun 2023 18:08:33 -0400
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13)
>
>
>
>
> >> Or maybe, I should leave `pp` alone and add a new `pp-region`
> >> function instead instead of combining two different calling conventions
> >> on a single function?
> >
> > That might be better, indeed, both for documentation and for usage.
>
> Done.  That also leads to fewer visible changes, so it's good (I just
> kept `pp-buffer` instead of adding `pp-region`).
>
> I pushed the result (after fixing the test problems with option B).
> Thanks,
>
>
>         Stefan
>
>
>
> --- End Message ---