diff options
author | Uri Lublin <uril@redhat.com> | 2016-12-06 18:06:29 +0200 |
---|---|---|
committer | Frediano Ziglio <fziglio@redhat.com> | 2016-12-06 16:45:07 +0000 |
commit | 4be1a6ec8c794f6847cb21f076f6f4dcbfb5862f (patch) | |
tree | 47aea064c10217c0bd7df800e37355ad0545f488 | |
parent | 547f9f438701653474c825d9addb2af7af7e1be2 (diff) |
red-record-qxl: add curly braces to empty while loop
Spice coding style suggests to use curly braces
for while blocks.
Some prefer the block to not be empty so continue
is untouched.
Signed-off-by: Uri Lublin <uril@redhat.com>
Acked-by: Frediano Ziglio <fziglio@redhat.com>
-rw-r--r-- | server/red-record-qxl.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/server/red-record-qxl.c b/server/red-record-qxl.c index 14a68a72..184d6b9a 100644 --- a/server/red-record-qxl.c +++ b/server/red-record-qxl.c @@ -847,8 +847,9 @@ static void child_output_setup(gpointer user_data) { int fd = GPOINTER_TO_INT(user_data); - while (dup2(fd, STDOUT_FILENO) < 0 && errno == EINTR) + while (dup2(fd, STDOUT_FILENO) < 0 && errno == EINTR) { continue; + } close(fd); // make sure file is not closed calling exec() @@ -890,8 +891,9 @@ RedRecord *red_record_new(const char *filename) fclose(f); spice_error("failed to setup filter for replay"); } - while (dup2(fd_in, fileno(f)) < 0 && errno == EINTR) + while (dup2(fd_in, fileno(f)) < 0 && errno == EINTR) { continue; + } close(fd_in); } |