Let's follow the 'standard' for optional components. This commit also
drops (now) unecessary #ifdef USE_LZ4 from lz4-encode.c, as the decision
to build this file is now made in Makefile.

Signed-off-by: Eduardo Lima (Etrunko) <[email protected]>
---
 server/Makefile.am   | 9 +++++++--
 server/lz4-encoder.c | 4 ----
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/server/Makefile.am b/server/Makefile.am
index 09710eb..0af8a1b 100644
--- a/server/Makefile.am
+++ b/server/Makefile.am
@@ -88,8 +88,6 @@ libserver_la_SOURCES =                                \
        inputs-channel-client.h         \
        jpeg-encoder.c                          \
        jpeg-encoder.h                          \
-       lz4-encoder.c                           \
-       lz4-encoder.h                           \
        main-channel.c                          \
        main-channel.h                          \
        main-channel-client.c                   \
@@ -154,6 +152,13 @@ libserver_la_SOURCES =                             \
        dcc-encoders.h                                  \
        $(NULL)
 
+if HAVE_LZ4
+libserver_la_SOURCES +=        \
+       lz4-encoder.c                           \
+       lz4-encoder.h                           \
+       $(NULL)
+endif
+
 if HAVE_SMARTCARD
 libserver_la_SOURCES +=        \
        smartcard.c             \
diff --git a/server/lz4-encoder.c b/server/lz4-encoder.c
index 7b6359f..f193fd8 100644
--- a/server/lz4-encoder.c
+++ b/server/lz4-encoder.c
@@ -18,8 +18,6 @@
 #include <config.h>
 #endif
 
-#ifdef USE_LZ4
-
 #define SPICE_LOG_DOMAIN "SpiceLz4Encoder"
 
 #include <arpa/inet.h>
@@ -119,5 +117,3 @@ int lz4_encode(Lz4EncoderContext *lz4, int height, int 
stride, uint8_t *io_ptr,
 
     return out_size;
 }
-
-#endif // USE_LZ4
-- 
2.5.5

_______________________________________________
Spice-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to