Re: [PATCH spice-server] Sort include order in source files

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

 



> 
> On 12/08/2016 08:32 PM, Frediano Ziglio wrote:
> > Sort based on coding style.
> >
> > Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> > ---
> >  server/inputs-channel.c |  2 +-
> >  server/jpeg-encoder.c   |  4 +++-
> >  server/mjpeg-encoder.c  |  8 +++++---
> >  server/reds-stream.c    | 13 +++++++------
> >  server/spicevmc.c       |  6 +++---
> >  server/tree.c           |  2 +-
> >  server/zlib-encoder.c   |  3 ++-
> >  7 files changed, 22 insertions(+), 16 deletions(-)
> >
> > diff --git a/server/inputs-channel.c b/server/inputs-channel.c
> > index bea0ddf..3781b85 100644
> > --- a/server/inputs-channel.c
> > +++ b/server/inputs-channel.c
> > @@ -24,10 +24,10 @@
> >  #include <fcntl.h>
> >  #include <stddef.h> // NULL
> >  #include <errno.h>
> > +#include <stdbool.h>
> >  #include <spice/macros.h>
> >  #include <spice/vd_agent.h>
> >  #include <spice/protocol.h>
> > -#include <stdbool.h>
> >
> >  #include <common/marshaller.h>
> >  #include <common/messages.h>
> > diff --git a/server/jpeg-encoder.c b/server/jpeg-encoder.c
> > index 1d090b6..5c5edf1 100644
> > --- a/server/jpeg-encoder.c
> > +++ b/server/jpeg-encoder.c
> > @@ -18,9 +18,11 @@
> >  #include <config.h>
> >  #endif
> >
> > +#include <stdio.h>
> 
> Hi Frediano,
> 
> Looks good, but why did you add stdio.h ?
> (And another one in mjpeg-encoder.c)
> 
> Uri.
> 

jpeg headers require some definitions (they are not
self sufficient).

Frediano

> 
> > +#include <jpeglib.h>
> > +
> >  #include "red-common.h"
> >  #include "jpeg-encoder.h"
> > -#include <jpeglib.h>
> >
> >  typedef struct JpegEncoder {
> >      JpegEncoderUsrContext *usr;
> > diff --git a/server/mjpeg-encoder.c b/server/mjpeg-encoder.c
> > index f1f8c91..3578a5a 100644
> > --- a/server/mjpeg-encoder.c
> > +++ b/server/mjpeg-encoder.c
> > @@ -19,12 +19,14 @@
> >  #include <config.h>
> >  #endif
> >
> > +#include <stdio.h>
> > +#include <inttypes.h>
> > +#include <jerror.h>
> > +#include <jpeglib.h>
> > +
> >  #include "red-common.h"
> >  #include "video-encoder.h"
> >  #include "utils.h"
> > -#include <jerror.h>
> > -#include <jpeglib.h>
> > -#include <inttypes.h>
> >
> >  #define MJPEG_MAX_FPS 25
> >  #define MJPEG_MIN_FPS 1
> > diff --git a/server/reds-stream.c b/server/reds-stream.c
> > index 9896eab..2730549 100644
> > --- a/server/reds-stream.c
> > +++ b/server/reds-stream.c
> > @@ -19,12 +19,6 @@
> >  #include <config.h>
> >  #endif
> >
> > -#include "main-dispatcher.h"
> > -#include "red-common.h"
> > -#include <common/log.h>
> > -#include "reds-stream.h"
> > -#include "reds.h"
> > -
> >  #include <errno.h>
> >  #include <netdb.h>
> >  #include <unistd.h>
> > @@ -35,6 +29,13 @@
> >
> >  #include <openssl/err.h>
> >
> > +#include <common/log.h>
> > +
> > +#include "main-dispatcher.h"
> > +#include "red-common.h"
> > +#include "reds-stream.h"
> > +#include "reds.h"
> > +
> >  struct AsyncRead {
> >      RedsStream *stream;
> >      void *opaque;
> > diff --git a/server/spicevmc.c b/server/spicevmc.c
> > index 765d287..e51feea 100644
> > --- a/server/spicevmc.c
> > +++ b/server/spicevmc.c
> > @@ -27,6 +27,9 @@
> >  #include <string.h>
> >  #include <netinet/in.h> // IPPROTO_TCP
> >  #include <netinet/tcp.h> // TCP_NODELAY
> > +#ifdef USE_LZ4
> > +#include <lz4.h>
> > +#endif
> >
> >  #include <common/generated_server_marshallers.h>
> >
> > @@ -35,9 +38,6 @@
> >  #include "red-channel-client.h"
> >  #include "reds.h"
> >  #include "migration-protocol.h"
> > -#ifdef USE_LZ4
> > -#include <lz4.h>
> > -#endif
> >
> >  /* todo: add flow control. i.e.,
> >   * (a) limit the tokens available for the client
> > diff --git a/server/tree.c b/server/tree.c
> > index c0e125c..58f6c5c 100644
> > --- a/server/tree.c
> > +++ b/server/tree.c
> > @@ -19,9 +19,9 @@
> >  #endif
> >
> >  #include <spice/qxl_dev.h>
> > +
> >  #include "red-parse-qxl.h"
> >  #include "display-channel.h"
> > -
> >  #include "tree.h"
> >
> >  static const char *draw_type_to_str(uint8_t type)
> > diff --git a/server/zlib-encoder.c b/server/zlib-encoder.c
> > index 96a7b54..1579ed7 100644
> > --- a/server/zlib-encoder.c
> > +++ b/server/zlib-encoder.c
> > @@ -19,9 +19,10 @@
> >  #include <config.h>
> >  #endif
> >
> > +#include <zlib.h>
> > +
> >  #include "red-common.h"
> >  #include "zlib-encoder.h"
> > -#include <zlib.h>
> >
> >  struct ZlibEncoder {
> >      ZlibEncoderUsrContext *usr;
> >
> 
> 
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://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]