[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#12507: [debbugs-tracker] Processed: severity 12507 wishlist
From: |
Thierry Volpiatto |
Subject: |
bug#12507: [debbugs-tracker] Processed: severity 12507 wishlist |
Date: |
Sat, 29 Sep 2012 17:12:40 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.2.50 (gnu/linux) |
"Drew Adams" <drew.adams@oracle.com> writes:
>> > The question about visiting the file is still open, though.
>>
>> I am using now the patch I sent here yesterday and it works
>> really good, faster and do backups (numered) as expected.
>> Hope it will be applied here in emacs because it DTRT.
>>
>> I don't understand what is the problem with "visiting the file".
>> See in precedent post why it is not bad visiting the file.
>> In the special case of bookmark-write-file, it is really not the
>> problem.
>
> Your question is for Stefan.
>
> Your patch is equivalent to the change I proposed originally: just replace
> `write-region' with `write-file'.
No, this is ineficient too, you write twice the same data.
The important thing is writing directly to the buffer of file.
For the backup thing, yes it is similar, but with unneeded steps,
going straight to save-buffer is better and faster IMO (of course if you
have started writing data in the file buffer)
But the worst thing is the actual version with write-region:
Slow and backup broken.
--
Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997
bug#12507: [debbugs-tracker] Processed: severity 12507 wishlist, Thierry Volpiatto, 2012/09/29
bug#12507: Option `(bookmark-)version-control': Use :tag so doc string matches menu, Karl Fogel, 2012/09/30