unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Stefan Kangas <stefan@marxist.se>
To: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: Toomas Rosin <toomas@rosin.ee>, 15821@debbugs.gnu.org
Subject: bug#15821: a better M-SPC
Date: Tue, 21 Jan 2020 02:20:58 +0100	[thread overview]
Message-ID: <87a76hob9h.fsf@marxist.se> (raw)
In-Reply-To: <jwvy54o6nli.fsf-monnier+emacsbugs@gnu.org> (Stefan Monnier's message of "Sat, 16 Nov 2013 16:42:24 -0500")

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>> My version of M-SPC is an improvement over the original Emacs one in
>> two respects: it operates both horizontally and vertically, and it
>> takes a numeric argument.  See the docstring for details.
>
> I don't myself use M-SPC much and have never used its numeric argument,
> so I'd like to hear what other users have to say about your suggestion.

I'm a heavy user of `just-one-space' (M-SPC), so I will venture to
comment given the lack of replies over the years.  I will not comment
on the code, but only Stefan Monnier's summary of it, so apologies if
there's something that I've misunderstood.

I also want to say first thank you to Toomas for taking the time to
write up a suggestion aiming to improving Emacs.  As will be clear
below, I'm personally not too keen on this particular change, however.

Note first that the numeric argument is currently very easy to
understand: leave exactly ARG spaces.  Negative ARG means to leave
the absolute value of ARG spaces and delete empty lines.

> Reading your docstring, I see the following changes:
> 1- On a non-empty blank line, leave exactly ARG spaces on it (without
>    ARG, make it empty).

M-SPC has until now always left one space by default.  I think the
proposal would make for a very frustrating and confusing user
experience.  Changing it is also backwards incompatible.

> 2- On an empty line, delete all blank lines above and below it, leaving
>    ARG empty lines.

I don't see why we would want to change `just-one-space' to operate on
lines in this way.

I think it would make more sense to modify `delete-blank-lines' to
leave ARG empty lines with a prefix argument.  But that's a separate
feature request, in my opinion.

> 3- At the beginning or end of a non-blank line, delete all leading
>    resp. trailing whitespace, leaving ARG (by default, zero) spaces.

Same comment as above regarding zero spaces.

The rest is no change compared to what we have now, AFAICT.

> 4- Don't treat a negative ARG as meaning to also delete newlines.
>
> Point 4 sounds like an oversight (you don't yourself use that "negative
> ARG" feature, so you didn't bother to implement it), right?

I think one of the more useful aspects of M-SPC, that I personally use
all the time, and which we should most definitely keep.

> For points 1 to 3, the main issue I see with them is that some people
> apparently tend to like M-SPC so much that they hit M-SPC when they just
> want to insert a SPC (Richard mentioned doing that, recently), so
> changing M-SPC so that it sometimes finishes with no space at point can
> be an annoyance.

In summary, if I understand them correctly, I oppose the proposed
changes, which would in my opinion be a change for the worse.  They
would also be backwards compatible and almost certainly break existing
use patterns.  (I know they would for me.)

I therefore recommend to close this as wontfix.

Best regards,
Stefan Kangas





  reply	other threads:[~2020-01-21  1:20 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-06  9:36 bug#15821: a better M-SPC Toomas Rosin
2013-11-16 21:42 ` Stefan Monnier
2020-01-21  1:20   ` Stefan Kangas [this message]
2020-01-21  9:24     ` Andreas Schwab
2020-01-22  3:13     ` Richard Stallman
2020-02-28 23:21     ` Stefan Kangas

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87a76hob9h.fsf@marxist.se \
    --to=stefan@marxist.se \
    --cc=15821@debbugs.gnu.org \
    --cc=monnier@iro.umontreal.ca \
    --cc=toomas@rosin.ee \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).