fairland: Fuse sectx[], secty[] into sect[]
authorMarkus Armbruster <armbru@pond.sub.org>
Fri, 21 Aug 2020 11:42:51 +0000 (13:42 +0200)
committerMarkus Armbruster <armbru@pond.sub.org>
Sun, 17 Jan 2021 20:24:28 +0000 (21:24 +0100)
Signed-off-by: Markus Armbruster <armbru@pond.sub.org>
src/util/fairland.c

index e07f08868030a277612c0bb6f4255e56f3fad584..0b5f5ff002b7b1c8ece1a7007ca279a937891cc8 100644 (file)
@@ -222,6 +222,10 @@ static const char *outfile = DEFAULT_OUTFILE_NAME;
 #define new_x(newx) (((newx) + WORLD_X) % WORLD_X)
 #define new_y(newy) (((newy) + WORLD_Y) % WORLD_Y)
 
+struct xy {
+    coord x, y;
+};
+
 /*
  * Island sizes
  * isecs[i] is the size of the i-th island.
@@ -230,6 +234,12 @@ static int *isecs;
 
 static int *capx, *capy;       /* location of the nc capitals */
 
+/*
+ * Island sectors
+ * The i-th island's j-th sector is at sect[i][j].
+ */
+struct xy **sect;
+
 /*
  * Island at x, y
  * own[XYOFFSET(x, y)] is x,y's island number, -1 if water.
@@ -286,8 +296,6 @@ static unsigned short *distance;
 static int *bfs_queue;
 static int bfs_queue_head, bfs_queue_tail;
 
-static int **sectx, **secty;   /* the sectors for each continent */
-
 #define NUMTRIES 10            /* keep trying to grow this many times */
 
 static const char *numletter =
@@ -591,18 +599,12 @@ allocate_memory(void)
     closest = malloc(WORLD_SZ() * sizeof(*closest));
     distance = malloc(WORLD_SZ() * sizeof(*distance));
     bfs_queue = malloc(WORLD_SZ() * sizeof(*bfs_queue));
-    sectx = calloc(nc + ni, sizeof(int *));
-    secty = calloc(nc + ni, sizeof(int *));
     isecs = calloc(nc + ni, sizeof(int));
-    for (i = 0; i < nc; ++i) {
-       sectx[i] = calloc(sc, sizeof(int));
-       secty[i] = calloc(sc, sizeof(int));
-    }
-    for (i = nc; i < nc + ni; ++i) {
-       sectx[i] = calloc(is * 2, sizeof(int));
-       secty[i] = calloc(is * 2, sizeof(int));
-    }
-
+    sect = malloc((nc + ni) * sizeof(*sect));
+    for (i = 0; i < nc; i++)
+       sect[i] = malloc(sc * sizeof(**sect));
+    for (i = nc; i < nc + ni; i++)
+       sect[i] = malloc(is * 2 * sizeof(**sect));
 }
 
 static void
@@ -817,7 +819,7 @@ xzone_around_island(int c, int dist)
     int i;
 
     for (i = 0; i < isecs[c]; i++)
-       xzone_around_sector(c, sectx[c][i], secty[c][i], dist);
+       xzone_around_sector(c, sect[c][i].x, sect[c][i].y, dist);
 }
 
 /*
@@ -910,8 +912,8 @@ bfs_enqueue_island(int c)
     int i;
 
     for (i = 0; i < isecs[c]; i++) {
-       if (is_coastal(sectx[c][i], secty[c][i]))
-           bfs_enqueue(c, sectx[c][i], secty[c][i], 0);
+       if (is_coastal(sect[c][i].x, sect[c][i].y))
+           bfs_enqueue(c, sect[c][i].x, sect[c][i].y, 0);
     }
 }
 
@@ -1030,8 +1032,8 @@ add_sector(int c, int x, int y)
 
     assert(own[off] == -1);
     xzone_around_sector(c, x, y, c < nc ? di : DISTINCT_ISLANDS ? id : 0);
-    sectx[c][isecs[c]] = x;
-    secty[c][isecs[c]] = y;
+    sect[c][isecs[c]].x = x;
+    sect[c][isecs[c]].y = y;
     isecs[c]++;
     own[off] = c;
     adj_land_update(x, y);
@@ -1068,8 +1070,8 @@ grow_one_sector(int c)
     newx = newy = -1;
 
     for (i = 0; i < isecs[c]; i++) {
-       x = sectx[c][i];
-       y = secty[c][i];
+       x = sect[c][i].x;
+       y = sect[c][i].y;
        off = XYOFFSET(x, y);
 
        for (dir = DIR_FIRST; dir <= DIR_LAST; dir++) {
@@ -1249,8 +1251,8 @@ grow_islands(void)
                if (isecs[c + j] != secs) {
                    isecs[c + j]--;
                    assert(isecs[c + j] == secs);
-                   x = sectx[c + j][secs];
-                   y = secty[c + j][secs];
+                   x = sect[c + j][secs].x;
+                   y = sect[c + j][secs].y;
                    own[XYOFFSET(x, y)] = -1;
                    adj_land_update(x, y);
                }
@@ -1343,9 +1345,9 @@ elevate_land(void)
        i0 = c < nc ? 2 : 0;
        n = isecs[c] - i0;
        for (i = 0; i < i0; i++)
-           elev[XYOFFSET(sectx[c][i], secty[c][i])] = PLATMIN;
+           elev[XYOFFSET(sect[c][i].x, sect[c][i].y)] = PLATMIN;
        for (i = 0; i < n; i++)
-           off[i] = XYOFFSET(sectx[c][i0 + i], secty[c][i0 + i]);
+           off[i] = XYOFFSET(sect[c][i0 + i].x, sect[c][i0 + i].y);
        qsort(off, n, sizeof(*off), elev_cmp);
        delta = (double)(HIGHMIN - LANDMIN - 1) / (n - nm - 1);
        elevation = LANDMIN;