On 2023-01-12 at 18:39 +02, Eli Zaretskii <eliz@gnu.org> wrote:
Not "the current 'pp'", but the implementation for this optional
behavior.
This is optional behavior, but I would prefer if the performance impact of
enabling this optional behavior would be documented, e.g. in the variable
docstring and NEWS.29. It's disabled by default and will only be used by those
like me and Visuwesh who the documentation and news, so I'm fine with it being
almost unusably slow if that's documented. Though better performance if possible
would be nice of course.
On 2023-01-12 at 22:03 +0530, Visuwesh <visuweshm@gmail.com> wrote:
Personally, I always thought it would be best if the user facing
commands like pp-eval-sexp and friends alone respected the user option.
Sounds like a good idea. I definetly didn't expect that me personally setting
this option for myself would affect how lisp objects are serialized to disk in
external packages. Not sure if I would only enable it for interactive commands
and the like, in the emacs-world we are all hackers and hard to say what is
user-facing. People might write their own functions using pp on small s-exps,
and wonder why this setting isn't doing anything. Not sure what's the best
approach there
Maybe pp isn't meant to be used for doing anything that's not meant primarily
for human eyes, like serialization of lisp objects, maybe it's an error on
package maintainers that use it that way, but at least that could also somehow
be communicated more clearly to them.