diff options
author | John Keeping | 2017-04-24 19:38:34 +0100 |
---|---|---|
committer | John Keeping | 2017-10-03 19:19:34 +0100 |
commit | 3b485cc5422f800d142c7023295e82c0a1c10b19 (patch) | |
tree | c5c46cdf02c2cff650cc15342c3a5ced861c912c /ui-patch.c | |
parent | de297883385030f4bdcd2f5c3839d1187611b949 (diff) | |
download | cgit-3b485cc5422f800d142c7023295e82c0a1c10b19.tar.gz cgit-3b485cc5422f800d142c7023295e82c0a1c10b19.tar.bz2 cgit-3b485cc5422f800d142c7023295e82c0a1c10b19.zip |
cache: flush stdio before restoring FDs
As described in commit 2efb59e (ui-patch: Flush stdout after outputting
data, 2014-06-11), we need to ensure that stdout is flushed before
restoring the file descriptor when writing to the cache. It turns out
that it's not just ui-patch that is affected by this but also raw diff
which writes to stdout internally.
Let's avoid risking more places doing this by ensuring that stdout is
flushed after writing in fill_slot().
Signed-off-by: John Keeping <john@keeping.me.uk>
Diffstat (limited to 'ui-patch.c')
-rw-r--r-- | ui-patch.c | 2 |
1 files changed, 0 insertions, 2 deletions
@@ -92,6 +92,4 @@ void cgit_print_patch(const char *new_rev, const char *old_rev, | |||
92 | log_tree_commit(&rev, commit); | 92 | log_tree_commit(&rev, commit); |
93 | printf("-- \ncgit %s\n\n", cgit_version); | 93 | printf("-- \ncgit %s\n\n", cgit_version); |
94 | } | 94 | } |
95 | |||
96 | fflush(stdout); | ||
97 | } | 95 | } |