all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Dmitry Gutov <dmitry@gutov.dev>
To: Arthur Miller <arthur.miller@live.com>
Cc: emacs-devel@gnu.org
Subject: Re: Missing snprintf in ucrt mingw + vc-refresh in find-file hook?
Date: Mon, 12 Feb 2024 16:58:52 +0200	[thread overview]
Message-ID: <207528e2-6bec-436e-8868-8e7b707133f6@gutov.dev> (raw)
In-Reply-To: <DU2PR02MB1010972EC939E6322B3EBE41A96482@DU2PR02MB10109.eurprd02.prod.outlook.com>

On 12/02/2024 15:56, Arthur Miller wrote:
> Dmitry Gutov<dmitry@gutov.dev>  writes:
> 
>> On 12/02/2024 12:06, Arthur Miller wrote:
>>> Also, calling vc-refresh obviously results in a quite expensive call to git
>>> process everytime I open a file from git repo. Is it really a good default
>>> option? At least I interpret it so since emacs -q has vc-refresh in
>>> find-file-hook. Why do we even need vc-refresh by default in find-file-hook?
>> To have the mode-line indicator up-to-date, I imagine.
> (setq vc-handled-backends nil) solved it. I would prefer such an expensive
> feature to be opt-in not opt-out, or at least better advertised so I don't have
> to find by a chance what Emacs is doing in the background; thanks.

The fact of the matter is, it's relatively expensive only on certain 
platforms, but much less so on the main one that we keep in mind when 
developing Emacs (GNU/Linux).

You might be curious to hear that visiting a file controlled by Git 
under MS Windows was ever slower in the past, and yours truly spent some 
effort reducing the number of process calls we take to compute a file's 
status (about 10 years ago or so).

>>                                              I imagine.
> I would appreciate if you leave sarcasm out of your answers.

No sarcasm here. Your question is valid: why not defer the process calls 
until some operation actually requests the file's status.

The answer stems from the UI considerations. And if we show it in the 
mode-line, we have to update the file's VCS status eagerly.



  reply	other threads:[~2024-02-12 14:58 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-02-12 10:06 Missing snprintf in ucrt mingw + vc-refresh in find-file hook? Arthur Miller
2024-02-12 13:44 ` Dmitry Gutov
2024-02-12 13:56   ` Arthur Miller
2024-02-12 14:58     ` Dmitry Gutov [this message]
2024-02-12 16:49       ` Eli Zaretskii
2024-02-12 18:05         ` Dmitry Gutov
2024-02-12 19:15           ` Eli Zaretskii
2024-02-12 19:25             ` Dmitry Gutov
2024-02-12 19:34               ` Eli Zaretskii
2024-02-13  9:47       ` Arthur Miller
2024-02-13 13:36         ` Eli Zaretskii
2024-02-13 14:30           ` Arthur Miller
2024-02-13 21:26           ` Dmitry Gutov
2024-02-13 23:10             ` Arthur Miller
2024-02-14  3:42               ` Dmitry Gutov
2024-02-14 21:04                 ` Arthur Miller
2024-02-14 22:37                   ` Dmitry Gutov
2024-02-15 11:16                     ` Arthur Miller
2024-02-14 14:30             ` Eli Zaretskii
2024-02-14 16:36               ` Dmitry Gutov
2024-02-14 16:51                 ` Eli Zaretskii
2024-02-14 17:01                   ` Dmitry Gutov
2024-02-14 17:29                     ` Eli Zaretskii
2024-02-14 21:05                       ` Dmitry Gutov
2024-02-12 14:36 ` Eli Zaretskii
2024-02-13 10:44   ` Arthur Miller
2024-02-13 13:13     ` Eli Zaretskii
  -- strict thread matches above, loose matches on Subject: below --
2024-02-12 17:37 Angelo Graziosi
2024-02-13 10:49 ` Arthur Miller
2024-02-13 19:00 ` Arthur Miller
2024-02-13 20:01   ` Eli Zaretskii
2024-02-13 22:05     ` Arthur Miller
2024-02-14 14:45       ` Eli Zaretskii
2024-02-14 18:43         ` Arthur Miller
2024-02-13 21:26   ` Angelo Graziosi
2024-02-13 22:09     ` Arthur Miller
2024-02-13 22:21       ` Angelo Graziosi
2024-02-13 22:26         ` Arthur Miller

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=207528e2-6bec-436e-8868-8e7b707133f6@gutov.dev \
    --to=dmitry@gutov.dev \
    --cc=arthur.miller@live.com \
    --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.