dump_marks(): remove a redundant call to rollback_lock_file()
When commit_lock_file() fails, it now always calls rollback_lock_file() internally, so there is no need to call that function here. Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu> Reviewed-by: Jonathan Nieder <jrnieder@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Michael Haggerty committed
Oct 1, 2014 at 12:28 UTC
32c3ec258e8b12ba29ce591b09a300621dba9b3a
1 file changed
+1
-3
fast-import.c
+1
-3
@@ -1832,10 +1832,8 @@ static void dump_marks(void)
1832
}
1833
1834
if (commit_lock_file(&mark_lock)) {
1835
- int saved_errno = errno;
1836
- rollback_lock_file(&mark_lock);
1835
failure |= error("Unable to commit marks file %s: %s",
1838
- export_marks_file, strerror(saved_errno));
1836
+ export_marks_file, strerror(errno));
1837
return;
1838
}
1839
}