On Wed, Oct 21, 2015 at 2:26 PM, Frediano Ziglio <fziglio@xxxxxxxxxx> wrote: > >> >> From: Marc-André Lureau <marcandre.lureau@xxxxxxxxx> >> >> Group enums with their respective struct location. >> --- >> server/red_dispatcher.h | 97 >> ++++++++++++++++++++++++++++++++++++++++++++++++- >> server/red_worker.h | 93 +---------------------------------------------- >> 2 files changed, 96 insertions(+), 94 deletions(-) >> >> diff --git a/server/red_dispatcher.h b/server/red_dispatcher.h >> index 320b7e3..dbe65d1 100644 >> --- a/server/red_dispatcher.h >> +++ b/server/red_dispatcher.h >> @@ -18,10 +18,13 @@ >> #ifndef _H_RED_DISPATCHER >> #define _H_RED_DISPATCHER >> >> +#include <errno.h> >> +#include "spice_server_utils.h" > > Would be better to include unistd.h instead of spice_server_utils.h > Beside that I would ack it I also would go for a unistd.h include. Any of the other contents from spice_server_utils.h is used here. ACK with this change. _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel