all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: phillip.lord@newcastle.ac.uk (Phillip Lord)
To: Magnar Sveen <magnars@gmail.com>
Cc: Stefan Monnier <monnier@iro.umontreal.ca>, emacs-devel@gnu.org
Subject: Re: save-excursion and the mark
Date: Wed, 15 Apr 2015 16:16:53 +0100	[thread overview]
Message-ID: <87r3rlqv4q.fsf@newcastle.ac.uk> (raw)
In-Reply-To: <CALVcYyDs7d5awNBwgT3tFzZkuOikFgWLZYeHkzeVwsFXf9KpHw@mail.gmail.com> (Magnar Sveen's message of "Wed, 15 Apr 2015 04:14:04 +0000")


Magnar

While I can appreciate your point (oops, pun). But, while introducing a
new macro and effectively deprecating save-excursion would be safe, it
would be unlikely to have any real effects at all, simply because as you
say Emacs is very stable. That is, the vast majority of save-excursion
calls will never get changed, even if save-excursion is deprecated.

Changing "save-excursion" and introducing "save-mark-excursion" would be
another possibility (although the latter would have  the difficulties
that Stefan has described with, for example, activate-mark-hook.

Phil

Magnar Sveen <magnars@gmail.com> writes:

> There are millions of lines of elisp that you've never seen. Emacs is after
> all the eminently hackable editor. And you're making breaking changes to a
> core API out of a sense of cleanliness? You've now been made aware of one
> case where the change is harmful, and your conclusion is "this is the one
> case where it is harmful". No, it's the first case where it's been detected
> - and that's because expand-region is widely used by thousands of people -
> and thankfully it even has tests. People's code is going to break.
>
> You are maintaining an API with many thousands of users. Don't break it. If
> you must, deprecate it. Introduce save-point, and add "You probably want to
> use save-point" to the docstring of save-excursion.



  parent reply	other threads:[~2015-04-15 15:16 UTC|newest]

Thread overview: 61+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-23  4:44 save-excursion and the mark Stefan Monnier
2015-02-23 11:34 ` Oleh Krehel
2015-02-23 22:33   ` Stefan Monnier
2015-02-23 23:14     ` Oleh Krehel
2015-02-24  0:12       ` Artur Malabarba
2015-02-24  8:47         ` Oleh Krehel
2015-02-24  2:31   ` Alexis
2015-02-24  3:52     ` Stefan Monnier
2015-02-25  1:58       ` Yuri Khan
2015-02-24 15:37 ` Barry Warsaw
2015-02-24 19:27   ` Stefan Monnier
2015-02-24 20:11     ` Barry Warsaw
2015-02-24 20:49       ` Artur Malabarba
2015-02-25  3:01         ` Stefan Monnier
2015-02-25  3:25           ` Artur Malabarba
2015-02-25 16:31           ` Barry Warsaw
2015-02-25 16:42             ` Stefan Monnier
2015-02-25 17:17             ` Drew Adams
2015-02-25 18:30     ` Richard Stallman
2015-02-26 13:50       ` Stefan Monnier
2015-02-27  0:13         ` Richard Stallman
2015-02-27 13:49           ` Stefan Monnier
2015-02-27 23:46             ` Richard Stallman
2015-02-28 18:50               ` martin rudalics
2015-03-02  5:36                 ` Stefan Monnier
2015-03-02  5:35               ` Stefan Monnier
2015-03-02 21:05                 ` Richard Stallman
2015-03-03 16:34                   ` Stefan Monnier
2015-03-04 12:01                     ` Richard Stallman
2015-02-24 16:05 ` Andreas Röhler
2015-02-25  2:59   ` Stefan Monnier
2015-02-25 11:49     ` Andreas Röhler
2015-02-25 12:35       ` Ivan Shmakov
2015-02-25 13:05         ` Andreas Röhler
2015-02-25 14:00       ` Stefan Monnier
2015-02-25 17:13         ` Andreas Röhler
2015-02-25  9:18 ` Ivan Shmakov
2015-02-25 15:56   ` Eli Zaretskii
2015-04-06 18:45 ` Magnar Sveen
2015-04-07 21:37   ` Stefan Monnier
2015-04-08  1:57   ` Stefan Monnier
2015-04-14 16:08     ` Magnar Sveen
2015-04-14 20:16       ` Stefan Monnier
2015-04-15  4:14         ` Magnar Sveen
2015-04-15 12:53           ` Stefan Monnier
2015-04-15 13:49             ` Magnar Sveen
2015-04-15 14:59               ` Stefan Monnier
2015-04-18  1:02                 ` Daniel Colascione
2015-04-15 14:24           ` Richard Stallman
2015-04-15 15:16           ` Phillip Lord [this message]
2015-04-19 15:28         ` Oleh Krehel
2015-04-20  2:03           ` Stefan Monnier
2015-04-20  6:31         ` Andreas Röhler
2015-04-23 10:03 ` Frank Fischer
2015-05-28 17:42 ` Nicolas Richard
  -- strict thread matches above, loose matches on Subject: below --
2015-04-17 18:53 Lars Magne Ingebrigtsen
2015-04-18  4:26 ` Stefan Monnier
2015-04-18 12:30   ` Wolfgang Jenkner
2015-04-25 12:45   ` Lars Magne Ingebrigtsen
2015-04-25 13:23     ` Lars Magne Ingebrigtsen
2015-04-25 14:19       ` Artur Malabarba

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=87r3rlqv4q.fsf@newcastle.ac.uk \
    --to=phillip.lord@newcastle.ac.uk \
    --cc=emacs-devel@gnu.org \
    --cc=magnars@gmail.com \
    --cc=monnier@iro.umontreal.ca \
    /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.