[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#34858: indent-relative called with prefix calls tab-to-tab-stop
From: |
Alex |
Subject: |
bug#34858: indent-relative called with prefix calls tab-to-tab-stop |
Date: |
Thu, 14 Mar 2019 12:57:08 -0600 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Eli Zaretskii <eliz@gnu.org> writes:
>> From: Alex <agrambot@gmail.com>
>> Cc: Alex Branham <alex.branham@gmail.com>, 34858@debbugs.gnu.org
>> Date: Thu, 14 Mar 2019 10:45:32 -0600
>> >
>> > I think this is a documentation bug, as it doesn't describe the
>> > special case of "no previous nonblank line".
>>
>> I believe that the case of "no previous nonblank line" could be
>> considered as there being a single indent point at column 0, so if
>> FIRST-ONLY is non-nil, then `tab-to-tab-stop' shouldn't be called. How
>> about the following patch instead?
>
> I don't understand what you are saying here. The doc string says "no
> indent point _beyond_" the column where point starts, so assuming a
> single indent point at column zero doesn't change anything.
Sorry, I wasn't clear. I meant that the special case here could be
considered (trivially) equivalent to the case where the previous
non-blank line has a single indent point at column zero, and therefore
should be treated equivalently when FIRST-ONLY is non-nil
(`tab-to-tab-stop` should not be called if on or past the first indent
point, i.e., column 0).
The usage of `beyond' does not change the behaviour when the starting
column is on the first indent point, which outside of this special case
is doing nothing. Specifically, the `indent' branch of the cond is
normally executed when FIRST-ONLY is non-nil because `indent' is set
within the `re-search-backward' conditional to be equal to the starting
column; however, in this special case, `re-search-backward' returns nil
and so `indent' remains nil. My patch covers this case by also checking
for FIRST-ONLY in the cond.
- bug#34858: indent-relative called with prefix calls tab-to-tab-stop, Alex Branham, 2019/03/14
- bug#34858: indent-relative called with prefix calls tab-to-tab-stop, Eli Zaretskii, 2019/03/14
- bug#34858: indent-relative called with prefix calls tab-to-tab-stop, Alex Branham, 2019/03/14
- bug#34858: indent-relative called with prefix calls tab-to-tab-stop, Filipp Gunbin, 2019/03/20
- bug#34858: indent-relative called with prefix calls tab-to-tab-stop, Alex Branham, 2019/03/25
- bug#34858: indent-relative called with prefix calls tab-to-tab-stop, Eli Zaretskii, 2019/03/26
- bug#34858: indent-relative called with prefix calls tab-to-tab-stop, Alex Branham, 2019/03/26
- bug#34858: indent-relative called with prefix calls tab-to-tab-stop, Eli Zaretskii, 2019/03/26
bug#34858: indent-relative called with prefix calls tab-to-tab-stop, Alex, 2019/03/14