server: misc header cleanups

Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com>
Signed-off-by: Frediano Ziglio <fziglio@redhat.com>
Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com>
Acked-by: Jonathon Jongsma <jjongsma@redhat.com>

server/Makefile.am

@@ -117,6 +117,8 @@ libspice_server_la_SOURCES =			\
 	sw-canvas.h			\
 	sound.c				\
 	sound.h				\
+	spice-experimental.h			\
+	spice.h					\
 	stat.h					\
 	spicevmc.c				\
 	spice_timer_queue.c			\

server/dispatcher.h

@@ -18,8 +18,7 @@
 #ifndef DISPATCHER_H
 #define DISPATCHER_H
 
-#include <spice.h>
-#include "utils.h"
+#include "red-common.h"
 
 typedef struct Dispatcher Dispatcher;
 

server/glz-encoder-priv.h

@@ -18,6 +18,8 @@
 #ifndef GLZ_ENCODER_PRIV_H_
 #define GLZ_ENCODER_PRIV_H_
 
+#include "red-common.h"
+
 /* Interface for using the dictionary for encoding.
    Data structures are exposed for the encoder for efficiency
    purposes. */

server/glz-encoder.h

@@ -20,7 +20,7 @@
 
 /* Manging the lz encoding using a dictionary that is shared among encoders */
 
-#include <stdint.h>
+#include "red-common.h"
 #include "common/lz_common.h"
 #include "glz-encoder-dict.h"
 

server/memslot.c

@@ -21,7 +21,6 @@
 
 #include <inttypes.h>
 
-#include "red-common.h"
 #include "memslot.h"
 
 static unsigned long __get_clean_virt(RedMemSlotInfo *info, QXLPHYSICAL addr)

server/red-common.h

@@ -18,16 +18,25 @@
 #ifndef _H_RED_COMMON
 #define _H_RED_COMMON
 
-#include <spice/macros.h>
-#include <string.h>
+#include <glib.h>
 
-#include "common/mem.h"
-#include "common/spice_common.h"
-#include "common/messages.h"
+#include <errno.h>
+#include <stdbool.h>
+#include <stdint.h>
+#include <string.h>
+#include <unistd.h>
+
+#include <spice/macros.h>
+#include "common/log.h"
 #include "common/lz_common.h"
+#include "common/marshaller.h"
+#include "common/mem.h"
+#include "common/messages.h"
+#include "common/ring.h"
+#include "common/spice_common.h"
+#include "common/draw.h"
 
 #include "spice.h"
-
-#define SPICE_GNUC_VISIBLE __attribute__ ((visibility ("default")))
+#include "utils.h"
 
 #endif

server/red-dispatcher.h

@@ -18,8 +18,6 @@
 #ifndef _H_RED_DISPATCHER
 #define _H_RED_DISPATCHER
 
-#include <unistd.h>
-#include <errno.h>
 #include "red-channel.h"
 
 typedef struct RedDispatcher RedDispatcher;

server/red-worker.h

@@ -18,9 +18,6 @@
 #ifndef _H_REDWORKER
 #define _H_REDWORKER
 
-#include <unistd.h>
-#include <errno.h>
-#include "utils.h"
 #include "red-common.h"
 #include "red-dispatcher.h"
 #include "red-parse-qxl.h"

server/reds.c

@@ -58,7 +58,6 @@
 #include "agent-msg-filter.h"
 #include "inputs-channel.h"
 #include "main-channel.h"
-#include "red-common.h"
 #include "red-dispatcher.h"
 #include "main-dispatcher.h"
 #include "sound.h"

server/spice-bitmap-utils.h

@@ -18,7 +18,6 @@
 #ifndef SPICE_BITMAP_UTILS_H_
 #define SPICE_BITMAP_UTILS_H_
 
-#include <glib.h>
 #include "red-common.h"
 
 typedef enum {

server/sw-canvas.c

@@ -17,9 +17,6 @@
 #ifdef HAVE_CONFIG_H
 #include <config.h>
 #endif
-
-#include "common/spice_common.h"
-
 #include "sw-canvas.h"
 #define SW_CANVAS_IMAGE_CACHE
 #include "common/sw_canvas.c"

server/utils.h

@@ -18,11 +18,9 @@
 #ifndef UTILS_H_
 # define UTILS_H_
 
-#include <glib.h>
-#include <time.h>
+#include "red-common.h"
 
-#include "common/ring.h"
-#include "common/log.h"
+#define SPICE_GNUC_VISIBLE __attribute__ ((visibility ("default")))
 
 static inline void set_bit(int index, uint32_t *addr)
 {