[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#70617: 30.0.50; python-indent-shift-right indents left when line is
From: |
kobarity |
Subject: |
bug#70617: 30.0.50; python-indent-shift-right indents left when line is whitespace |
Date: |
Tue, 30 Apr 2024 16:26:13 +0900 |
User-agent: |
Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM-LB/1.14.9 (Gojō) APEL-LB/10.8 EasyPG/1.0.0 Emacs/30.0.50 (x86_64-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO) |
Eli Zaretskii wrote:
>
> > From: Patrick Poitras <patrick.f.poitras@gmail.com>
> > Date: Sat, 27 Apr 2024 16:16:46 -0700
> >
> > Hi all.
> >
> > I've noticed this weird behavior of python-indent-shift-right.
> >
> > Let's say I have a block of code. The "|" character represents my cursor
> >
> > In a python file, I write
> >
> > if True:
> > |
> >
> > (... file contents continue)
> >
> > and then call python-indent-shift-right on an empty line. It does nothing.
> > If the second line has any amount of
> > spaces, the whitespace is entirely deleted.
> >
> > Given that Python's whitespace is syntactically significant, the expected
> > behavior should be to add an indent
> > to an empty line, or to prepend 4 characters (or 8, given user choice) to
> > the empty line if it has whitespace.
> > From a cursory glance, I think we're running into some behavior of
> > indent-rigidly that tries to prevent the
> > addition of superfluous white space following an indented block of code. I
> > assume this would be potentially
> > desirable for languages with non-significant whitespace, but in the case of
> > the python-specific function, the
> > resulting behavior defies both the function name and description.
>
> Any comments?
Hi Patrick,
I believe this behavior is intended to avoid adding extra whitespaces
when processing a region containing blank lines. There may be an
approach to change to indent even a blank line as a special case when
the region is not active, but I personally think that Enter or TAB
indentation is sufficient in such cases.