summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorUri Lublin <uril@redhat.com>2016-12-06 18:06:29 +0200
committerFrediano Ziglio <fziglio@redhat.com>2016-12-06 16:45:07 +0000
commit4be1a6ec8c794f6847cb21f076f6f4dcbfb5862f (patch)
tree47aea064c10217c0bd7df800e37355ad0545f488
parent547f9f438701653474c825d9addb2af7af7e1be2 (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.c6
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);
}