Mercurial > vim

changeset 25951:560132056b97 v8.2.3509

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
patch 8.2.3509: undo file is not synced Commit: https://github.com/vim/vim/commit/340dd0fbe462a15a9678cfba02085b4adcc45f02 Author: Bram Moolenaar <Bram@vim.org> Date: Thu Oct 14 17:52:23 2021 +0100 patch 8.2.3509: undo file is not synced Problem: Undo file is not synced. (Sami Farin) Solution: Sync the undo file if 'fsync' is set. (Christian Brabandt, closes #8879, closes #8920)
author Bram Moolenaar <Bram@vim.org>
date Thu, 14 Oct 2021 19:00:07 +0200
parents 8b36e7633cee
children 7eded0585ee0
files runtime/doc/options.txt src/undo.c src/version.c
diffstat 3 files changed, 9 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/runtime/doc/options.txt
+++ b/runtime/doc/options.txt
@@ -3610,8 +3610,8 @@ A jump table for the options with a shor
 	systems without an fsync() implementation, this variable is always
 	off.
 	Also see 'swapsync' for controlling fsync() on swap files.
-	'fsync' also applies to |writefile()|, unless a flag is used to
-	overrule it.
+	'fsync' also applies to |writefile()| (unless a flag is used to
+	overrule it) and when writing undo files (see |undo-persistence|).
 	This option cannot be set from a |modeline| or in the |sandbox|, for
 	security reasons.
 
--- a/src/undo.c
+++ b/src/undo.c
@@ -1786,6 +1786,11 @@ u_write_undo(
 	write_ok = FALSE;
 #endif
 
+#if defined(UNIX) && defined(HAVE_FSYNC)
+    if (p_fs && fflush(fp) == 0 && vim_fsync(fd) != 0)
+	write_ok = FALSE;
+#endif
+
 write_error:
     fclose(fp);
     if (!write_ok)
--- a/src/version.c
+++ b/src/version.c
@@ -758,6 +758,8 @@ static char *(features[]) =
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    3509,
+/**/
     3508,
 /**/
     3507,