all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: martin rudalics <rudalics@gmx.at>
To: Juanma Barranquero <lekktu@gmail.com>
Cc: 21766@debbugs.gnu.org, Markus Triska <triska@metalevel.at>
Subject: bug#21766: 25.0.50; delete-trailing-whitespace sometimes deletes non-whitespace
Date: Tue, 27 Oct 2015 09:25:15 +0100	[thread overview]
Message-ID: <562F34EB.8030209@gmx.at> (raw)
In-Reply-To: <CAAeL0SSefTPd6Z9A8XpJJ_axkBd2aS6EX-f4mz_Cry-6wx=wUQ@mail.gmail.com>

 > Hmm, what's the gain for the added complexity?

Maybe none.  I have no idea how the regexp machine works when looking
for "\\s-$".  If it looks for whitespace followed by a line end and not
for whitespace preceding a line end, the old version will be slow with
indented code.

martin





  reply	other threads:[~2015-10-27  8:25 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-26 23:07 bug#21766: 25.0.50; delete-trailing-whitespace sometimes deletes non-whitespace Markus Triska
2015-10-27  0:15 ` Juanma Barranquero
2015-10-27  7:07   ` Andreas Röhler
2015-10-27  7:53     ` Juanma Barranquero
2015-10-27  8:05   ` martin rudalics
2015-10-27  8:12     ` Juanma Barranquero
2015-10-27  8:25       ` martin rudalics [this message]
2015-10-27  8:32         ` Juanma Barranquero
2015-10-27  9:01   ` Andreas Schwab
2015-10-27  9:16     ` Juanma Barranquero
2015-10-27  9:24       ` martin rudalics
2015-10-27  9:46       ` Andreas Schwab
2015-10-27 10:21         ` Juanma Barranquero
2015-10-27 16:03           ` Juanma Barranquero
2015-10-27 16:03           ` Juanma Barranquero
2015-10-28  1:51             ` Stephen Leake
2015-10-28  1:51             ` Stephen Leake
2015-10-28  9:08               ` Juanma Barranquero
2015-10-28 17:38                 ` Juanma Barranquero
2015-10-28  9:08               ` Juanma Barranquero
2015-10-28 19:19         ` Stefan Monnier
2015-10-29  6:30           ` Andreas Röhler
2015-10-29  9:41             ` Juanma Barranquero

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

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

  git send-email \
    --in-reply-to=562F34EB.8030209@gmx.at \
    --to=rudalics@gmx.at \
    --cc=21766@debbugs.gnu.org \
    --cc=lekktu@gmail.com \
    --cc=triska@metalevel.at \
    /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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.