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@xxxxxxxxxx> --- server/red-record-qxl.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/server/red-record-qxl.c b/server/red-record-qxl.c index 14a68a7..184d6b9 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); } -- 2.9.3 _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel