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: Eli Zaretskii
Subject: bug#58665: Indentation in lisp/emacs-lisp/syntax.el is wrong
Date: Sun, 03 Sep 2023 13:39:19 +0300

> From: Stefan Kangas <stefankangas@gmail.com>
> Date: Sun, 3 Sep 2023 03:21:33 -0700
> Cc: Eli Zaretskii <eliz@gnu.org>, 58665@debbugs.gnu.org
> 
> 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.

If you think this change could be useful, I won't object applying it.





reply via email to

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