GNU bug report logs - #35036
27.0.50; delete-indentation removes all blank lines

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 35036 in the body.
You can then email your comments to 35036 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Alex Branham <alex.branham <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 27.0.50; delete-indentation removes all blank lines
Date: Thu, 28 Mar 2019 16:19:18 -0500

After 8fa94a1ecc18a41ca2738f438b3fbc817c9fdc82, delete-indentation
removes all blank lines so that a buffer:

one


two|


with | as point, M-x delete-indentation results in:

one two

whereas before it would result in:

one

two

This seems contrary to the documentation, which states, "Join this line
to previous and fix up whitespace at join." and also the NEWS entry,
"When there's no active region, the command works on the current and the
previous or the next line, as before."

I'm happy to submit a patch fixing this, but should it change the
documentation to match the new behavior, or fix the code so that the
behavior is unchanged?

Also, is the patch in Bug#35021 going to get applied soonish? If so I'll
hold off in order not to create conflicts...

Thanks,
Alex

Message #8 received at 35036 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Alex Branham <alex.branham <at> gmail.com>
Cc: 35036 <at> debbugs.gnu.org
Subject: Re: bug#35036: 27.0.50; delete-indentation removes all blank lines
Date: Fri, 29 Mar 2019 11:56:45 +0300

merge 35036 35021
thanks

> From: Alex Branham <alex.branham <at> gmail.com>
> Date: Thu, 28 Mar 2019 16:19:18 -0500
> 
> After 8fa94a1ecc18a41ca2738f438b3fbc817c9fdc82, delete-indentation
> removes all blank lines so that a buffer:
> 
> one
> 
> 
> two|
> 
> 
> with | as point, M-x delete-indentation results in:
> 
> one two
> 
> whereas before it would result in:
> 
> one
> 
> two
> 
> This seems contrary to the documentation, which states, "Join this line
> to previous and fix up whitespace at join." and also the NEWS entry,
> "When there's no active region, the command works on the current and the
> previous or the next line, as before."

But you did have a region, albeit not active, right?  Because if mark
wasn't set, you'd have an error message.

So I believe this is a duplicate of sorts of bug#35021, and should be
fixed together with that bug.

> Also, is the patch in Bug#35021 going to get applied soonish? If so I'll
> hold off in order not to create conflicts...

Soonish, yes.  Stay tuned.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.