Complete.Org: Mailing Lists: Archives: freeciv-dev: October 2003:
[Freeciv-Dev] (PR#6442) gen5 wrapping
Home

[Freeciv-Dev] (PR#6442) gen5 wrapping

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: undisclosed-recipients: ;
Subject: [Freeciv-Dev] (PR#6442) gen5 wrapping
From: "Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>
Date: Wed, 8 Oct 2003 12:18:14 -0700
Reply-to: rt@xxxxxxxxxxxxxx

This simple patch replaces two gen5 wrapping constants with queried 
topology values.

jason

Index: server/mapgen.c
===================================================================
RCS file: /home/freeciv/CVS/freeciv/server/mapgen.c,v
retrieving revision 1.118
diff -u -r1.118 mapgen.c
--- server/mapgen.c     2003/10/03 11:29:31     1.118
+++ server/mapgen.c     2003/10/08 19:17:25
@@ -2099,9 +2099,6 @@
 **************************************************************************/
 static void mapgenerator5(void)
 {
-  const bool xnowrap = FALSE;  /* could come from topology */
-  const bool ynowrap = TRUE;   /* could come from topology */
-
   /* 
    * How many blocks should the x and y directions be divided into
    * initially. 
@@ -2109,11 +2106,11 @@
   const int xdiv = 6;          
   const int ydiv = 5;
 
-  int xdiv2 = xdiv + (xnowrap ? 1 : 0);
-  int ydiv2 = ydiv + (ynowrap ? 1 : 0);
+  int xdiv2 = xdiv + (topo_has_flag(TF_WRAPX) ? 0 : 1);
+  int ydiv2 = ydiv + (topo_has_flag(TF_WRAPY) ? 0 : 1);
 
-  int xmax = map.xsize - (xnowrap ? 1 : 0);
-  int ymax = map.ysize - (ynowrap ? 1 : 0);
+  int xmax = map.xsize - (topo_has_flag(TF_WRAPX) ? 0 : 1);
+  int ymax = map.ysize - (topo_has_flag(TF_WRAPY) ? 0 : 1);
   int x, y, minval;
   /* just need something > log(max(xsize, ysize)) for the recursion */
   int step = map.xsize + map.ysize; 
@@ -2136,7 +2133,7 @@
 
   /* if we aren't wrapping stay away from edges to some extent, try
      even harder to avoid the edges naturally if separatepoles is true */
-  if (xnowrap) {
+  if (!topo_has_flag(TF_WRAPX)) {
     for (y = 0; y < ydiv2; y++) {
       hmap(0, y * ymax / ydiv) -= avoidedge;
       hmap(xmax, y * ymax / ydiv) -= avoidedge;
@@ -2149,7 +2146,7 @@
     }
   }
 
-  if (ynowrap) {
+  if (!topo_has_flag(TF_WRAPY)) {
     for (x = 0; x < xdiv2; x++) {
       hmap(x * xmax / xdiv, 0) -= avoidedge;
       hmap(x * xmax / xdiv, ymax) -= avoidedge;

[Prev in Thread] Current Thread [Next in Thread]