GNU bug report logs - #78340
[PATCH] New option for comp *.eln file name by the file timestamp of *.el

Previous Next

Full log

Message #62 received at 78340 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Lin Sun <sunlin7 <at> hotmail.com>
Cc: acorallo <at> gnu.org, owinebar <at> gmail.com, 78340 <at> debbugs.gnu.org
Subject: Re: bug#78340: [PATCH] New option for comp *.eln file name by the
 file timestamp of *.el
Date: Mon, 12 May 2025 22:11:02 +0300

> From: Lin Sun <sunlin7 <at> hotmail.com>
> CC: "acorallo <at> gnu.org" <acorallo <at> gnu.org>, "78340 <at> debbugs.gnu.org"
> 	<78340 <at> debbugs.gnu.org>
> Date: Mon, 12 May 2025 16:38:06 +0000
> msip_labels:
> 
> From: Eli Zaretskii <eliz <at> gnu.org>
> Sent: Monday, May 12, 2025 02:24 PM
> > ... which will probably be slow on Windows
> > for the same reason computing the content hash is slow...
> 
> I want highlight reading a .el file will call windows api OpenFile/ReadFile.../CloseFile.
> Here is an example that Emacs try to read the .el file to calculate the .eln name, it had called "ReadFile()" 6 time to read full content.

Yes, because Emacs reads files in chunks of 32KB.

> If Emacs uses timestamp, just call Getattribute() function, will reduce counts on windows file api calling.

I tried to explain why relying on time stamps is not reliable enough
in this case, it brings too high risk of loading an incompatible .eln
file.

In Emacs, we value correctness before speed.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.