[debbugs-tracker] bug#35036: closed (27.0.50; delete-indentation removes all blank lines)

[GNU bug Tracking System]

Your message dated Sun, 31 Mar 2019 17:42:43 +0100
with message-id <address@hidden>
and subject line Re: bug#35021: M-^ (delete-indentation) doesn't work without a 
mark present
has caused the debbugs.gnu.org bug report #35021,
regarding 27.0.50; delete-indentation removes all blank lines
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
35021: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=35021
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> 27.0.50; delete-indentation removes all blank lines
>
>
>
>
> Date: 
>
> Thu, 28 Mar 2019 16:19:18 -0500
>
>
>
>
> User-agent: 
>
> mu4e 1.1.0; emacs 27.0.50
>
>
>
>
> 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
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#35021: M-^ (delete-indentation) doesn't work without a mark present
>
>
>
>
> Date: 
>
> Sun, 31 Mar 2019 17:42:43 +0100
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux)
>
>
>
>
> Eli Zaretskii <address@hidden> writes:
>
> >> From: "Basil L. Contovounesios" <address@hidden>
> >> Cc: Alex Branham <address@hidden>,  <address@hidden>,  <address@hidden>,  
> >> <address@hidden>,  <address@hidden>
> >> Date: Sun, 31 Mar 2019 03:41:47 +0100
> >> 
> >> > Looks OK to me, please push.
> >> 
> >> Thanks, but my last patch only fixed the bug with the interactive spec,
> >> and I have since discovered and fixed a few more edge-cases (including
> >> that reported by Alex in bug#35036).
> >> 
> >> I attach the updated patch.  Can I push this one instead?
> >
> > Yes, this LGTM as well.
>
> Thanks, I pushed to master and am thus closing this report.
>
> -- 
> Basil
>
>
> --- End Message ---