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

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

bug#58665: Indentation in lisp/emacs-lisp/syntax.el is wrong


From: Stefan Kangas
Subject: bug#58665: Indentation in lisp/emacs-lisp/syntax.el is wrong
Date: Sun, 3 Sep 2023 03:21:33 -0700

Jim Blandy <jblandy@mozilla.com> writes:

> This code changes almost never. It's been like that since 2016. It
> doesn't make sense to let it sit there with formatting that obscures
> the structure of the code.
>
> On Thu, Oct 20, 2022 at 10:57 AM Eli Zaretskii <eliz@gnu.org> wrote:
>
>  > From: Jim Blandy <jblandy@mozilla.com>
>  > Date: Thu, 20 Oct 2022 10:23:30 -0700
>  >
>  > 2022-10-20  Jim Blandy  <jimb@red-bean.com>
>  >
>  >         * lisp/emacs-lisp/syntax.el: Fix indentation in `syntax-ppss'.
>
>  Thanks, but we don't like pure-whitespace changes.  We fix these
>  issues when we change code around the places with wrong indentation.

I agree that we should avoid making whitespace-only changes in general.
Jim makes the point that readability counts.  He also points out that
this code changes so infrequently that we do not need to be overly
worried about any annoying merge conflicts.  I think these are valid
points.

So on balance, I'd be willing to make an exception and take this patch.
If Eli is still not convinced, I think it would be better to close this
bug and move on.  I think we can all agree it's not worth spending
energy discussing it further.





reply via email to

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