all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Eric S. Raymond" <esr@thyrsus.com>
To: David Kastrup <dak@gnu.org>
Cc: emacs-devel@gnu.org
Subject: Re: vc-state-heuristic is gone
Date: Mon, 24 Nov 2014 04:49:29 -0500	[thread overview]
Message-ID: <20141124094929.GA32148@thyrsus.com> (raw)
In-Reply-To: <87zjbh3r98.fsf@fencepost.gnu.org>

David Kastrup <dak@gnu.org>:
> If I get a VC call across an ssh link for every keystroke I make in some
> Tramp-edited buffer because the modeline wants to display the non-cached
> current branch I am on, this is not going to lead to happy editing.

Why do you suppose this could ever occur?  There is not and never has been 
any VC code that fires on an ordinary keystroke - that is, one that is not
bound as a VC command.

The branch field, in particular, would only have to be updated when a
VC operation that could potentially change it is invoked. My guess is
that you have "removing caching" somewhat confused with
"bulletproofing VC against operations going on behind its back".

Removing the ancient caching code is partial hardening against those -
it does eliminate *some* TOCTOU conditions, which as I previously
noted is one reason it's a good idea. But the main goal was to reduce
the complexity of the mode code, not futilely attempt to prevent
*every* potential way that an out-of-VC operation could invalidate
VC's stored state.  That would be a crazy, infeasably expensive thing
to do, and I'm certainly not going to try it.

What my changes do is make the old modes that used to cache a lot of
state approximately as safe as the modern ones that don't, like
git's. That is, VC will behave reasonably in the face of operations
done outside it context *through the ordinary VCS commands*. 

That is, your branch tag display might get stale, but that would be
about the worst of it; VC operations made from an incorrect state will
generally error out harmlessly rather than doing something nasty.
-- 
		<a href="http://www.catb.org/~esr/">Eric S. Raymond</a>



  reply	other threads:[~2014-11-24  9:49 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-23 21:56 vc-state-heuristic is gone Unknown
2014-11-24  8:09 ` David Kastrup
2014-11-24  8:33   ` Eric S. Raymond
2014-11-24  9:02     ` David Kastrup
2014-11-24  9:49       ` Eric S. Raymond [this message]
2014-11-24 10:35         ` David Kastrup
2014-11-24 10:46           ` Eric S. Raymond
2014-11-24 11:57             ` Michael Albinus
2014-11-24 13:03               ` Testing the new VC code Eric S. Raymond
2014-11-24 18:35                 ` Steinar Bang
2014-11-24 19:05                   ` Eric S. Raymond
2014-11-25  1:52                   ` Yuri Khan
2014-11-25  2:52                     ` Eric S. Raymond
2014-11-25  4:25                       ` Stefan Monnier
2014-11-25  8:31                         ` Steinar Bang
2014-11-25  8:43                           ` Steinar Bang
2014-11-25  9:09                           ` David Kastrup
2014-11-25  9:12                             ` Steinar Bang
2014-11-25 10:03                               ` Andreas Schwab
2014-11-24 19:39                 ` Michael Albinus
2014-11-25  2:50                   ` Eric S. Raymond
2014-11-25  6:44                     ` David Kastrup
2014-11-25 13:22                       ` Ergonomics and neurology for interface designers Eric S. Raymond
2014-11-25 14:15                         ` David Kastrup
2014-11-25 21:29                           ` Eric S. Raymond
2014-11-25 16:32                       ` Testing the new VC code Stephen J. Turnbull
2014-11-25 12:12                     ` Michael Albinus
2014-11-25 13:42                       ` Eric S. Raymond
2014-11-25 14:55                         ` Michael Albinus
2014-11-25 21:17                           ` Eric S. Raymond
2014-11-26 10:12                             ` Michael Albinus
2014-11-26 11:46                               ` Eric S. Raymond
2014-11-25 14:35                       ` Stefan Monnier
2014-11-25 21:25                         ` Eric S. Raymond
2014-11-24 14:40     ` vc-state-heuristic is gone Stefan Monnier
2014-11-24 16:17 ` Richard Stallman
2014-11-25  5:15   ` Karl Fogel
2014-11-25  5:47     ` Eric S. Raymond
2014-11-24 16:17 ` Richard Stallman
2014-11-24 17:52   ` Eric S. Raymond
2014-11-25 13:20     ` Richard Stallman

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=20141124094929.GA32148@thyrsus.com \
    --to=esr@thyrsus.com \
    --cc=dak@gnu.org \
    --cc=emacs-devel@gnu.org \
    /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.