<URL: http://bugs.freeciv.org/Ticket/Display.html?id=39926 >

William Allen Simpson wrote:
> For portability, we should re-write this section.
> 
Damnit, was already re-written in 2.1, but not in 2.2/trunk.  As I've
said before, I really really really wish that folks fixing bugs in 2.1
would also fix them in 2.2/trunk....

So, I fixed some reserved words in the macro parameters, too.

Committed S2_2 revision 14094.
Committed trunk revision 14095.

Index: server/settings.c
===================================================================
--- server/settings.c   (revision 14094)
+++ server/settings.c   (working copy)
@@ -195,28 +195,49 @@
   return TRUE;
 }
 
+
+/************************************************************************/
+#if defined(HAVE_LIBBZ2)
+#define GAME_MIN_COMPRESS_TYPE FZ_PLAIN
+#define GAME_MAX_COMPRESS_TYPE FZ_BZIP2
+#define GAME_DEFAULT_COMPRESS_TYPE FZ_BZIP2
+
+#elif defined(HAVE_LIBZ)
+#define GAME_MIN_COMPRESS_TYPE FZ_PLAIN
+#define GAME_MAX_COMPRESS_TYPE FZ_ZLIB
+#define GAME_DEFAULT_COMPRESS_TYPE FZ_ZLIB
+
+#else
+#define GAME_MIN_COMPRESS_TYPE FZ_PLAIN
+#define GAME_MAX_COMPRESS_TYPE FZ_PLAIN
+#define GAME_DEFAULT_COMPRESS_TYPE FZ_PLAIN
+
+#endif
+
+/************************************************************************/
+
 #define GEN_BOOL(name, value, sclass, scateg, slevel, to_client,       \
-                short_help, extra_help, func, default)                 \
+                short_help, extra_help, func, _default)                \
   {name, sclass, to_client, short_help, extra_help, SSET_BOOL,         \
-      scateg, slevel, &value, default, func,                           \
+      scateg, slevel, &value, _default, func,                          \
       NULL, 0, NULL, 0, 0,                                             \
       NULL, NULL, NULL, 0},
 
 #define GEN_INT(name, value, sclass, scateg, slevel, to_client,                
\
-               short_help, extra_help, func, min, max, default)        \
+               short_help, extra_help, func, _min, _max, _default)     \
   {name, sclass, to_client, short_help, extra_help, SSET_INT,          \
       scateg, slevel,                                                  \
       NULL, FALSE, NULL,                                               \
-      &value, default, func, min, max,                                 \
+      &value, _default, func, _min, _max,                              \
       NULL, NULL, NULL, 0},
 
 #define GEN_STRING(name, value, sclass, scateg, slevel, to_client,     \
-                  short_help, extra_help, func, default)               \
+                  short_help, extra_help, func, _default)              \
   {name, sclass, to_client, short_help, extra_help, SSET_STRING,       \
       scateg, slevel,                                                  \
       NULL, FALSE, NULL,                                               \
       NULL, 0, NULL, 0, 0,                                             \
-      value, default, func, sizeof(value)},
+      value, _default, func, sizeof(value)},
 
 #define GEN_END                                                        \
   {NULL, SSET_LAST, SSET_SERVER_ONLY, NULL, NULL, SSET_INT,    \
@@ -976,7 +997,6 @@
          N_("If non-zero, saved games will be compressed using zlib "
             "(gzip format) or bzip2. Larger values will give better "
             "compression but take longer."), NULL,
-
          GAME_MIN_COMPRESS_LEVEL, GAME_MAX_COMPRESS_LEVEL,
          GAME_DEFAULT_COMPRESS_LEVEL)
 
@@ -988,13 +1008,8 @@
              " 1 - zlib (gzip format)\n"
              " 2 - bzip2\n"
              "Not all servers support all compression methods."), NULL,
-#if !defined(HAVE_LIBBZ2) && !defined(HAVE_LIBZ)
-          FZ_PLAIN, FZ_PLAIN, FZ_PLAIN)
-#elif !defined(HAVE_LIBBZ2) && defined(HAVE_LIBZ)
-          FZ_PLAIN, FZ_ZLIB, FZ_ZLIB)
-#else
-          FZ_PLAIN, FZ_BZIP2, FZ_BZIP2)
-#endif
+         GAME_MIN_COMPRESS_TYPE, GAME_MAX_COMPRESS_TYPE,
+         GAME_DEFAULT_COMPRESS_TYPE)
 
   GEN_STRING("savename", game.save_name,
             SSET_META, SSET_INTERNAL, SSET_VITAL, SSET_SERVER_ONLY,
_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to