ref: 8ed65078dab6a5832b857de46c5d461c26e46746
parent: 7311fc9ef81bcab931b9ca768f018a627f281933
author: ISSOtm <[email protected]>
date: Tue Mar 22 14:38:23 EDT 2022
Import fixes to random image generator
--- a/test/randtilegen.c
+++ b/test/randtilegen.c
@@ -65,13 +65,13 @@
}
static void generate_random_image(png_structp png, png_infop pngInfo) {
-#define NB_TILES 100
+#define NB_TILES 10 * 10
struct {
unsigned char palette;
unsigned char nbColors;
} attributes[NB_TILES];
uint8_t tileData[NB_TILES][8][8];
- // These two are in tiles, not pixels
+ // These two are in tiles, not pixels, and in range [3; 10], hence `NB_TILES` above
// Both width and height are 4-bit values, so nbTiles is 8-bit (OK!)
uint8_t const width = getRandomBits(3) + 3, height = getRandomBits(3) + 3,
nbTiles = width * height;
@@ -86,6 +86,7 @@
static uint8_t const popcount[] = {1, 1, 2, 1, 2, 2, 3, 1, 2, 2, 3, 2, 3, 3, 4,
1, 2, 2, 3, 2, 3, 3, 4, 2, 3, 3, 4, 3, 4};
attributes[p].nbColors = popcount[pal - 1];
+ // Handle single-color tiles the simple way, without trying to pull more random bits
if (attributes[p].nbColors < 2) {
memset(tileData[p], 0, sizeof(tileData[p]));
continue;
@@ -104,6 +105,9 @@
for (index = 0; total; index++) {
if (attributes[index].nbColors == attributes[p].nbColors) {
total--;
+ }
+ if (!total) {
+ index--;
}
}
} else {