Re: [PATCH 1/9] server/dispatcher: move worker enums to dispatcher header

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 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.
> 

Thanks, merged

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]