Spice coding style suggests to use curly braces for while blocks. Signed-off-by: Uri Lublin <uril@xxxxxxxxxx> --- server/red-record-qxl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/server/red-record-qxl.c b/server/red-record-qxl.c index 8af2c9c..adc487b 100644 --- a/server/red-record-qxl.c +++ b/server/red-record-qxl.c @@ -846,8 +846,8 @@ static void child_output_setup(gpointer user_data) { int fd = GPOINTER_TO_INT(user_data); - while (dup2(fd, STDOUT_FILENO) < 0 && errno == EINTR) - continue; + while (dup2(fd, STDOUT_FILENO) < 0 && errno == EINTR) { + } close(fd); // make sure file is not closed calling exec() -- 2.7.4 _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel