Merge pull request #572 from red031000/master

red031000 · web-flow · commit cd055413b0a1 · 2025-11-12T06:47:04.000Z
nitrogfx - fix clang warning
diff --git a/tools/nitrogfx/gfx.c b/tools/nitrogfx/gfx.c
@@ -126,7 +126,7 @@ static void Convert8BppFrom4BppTiles(unsigned char *src, unsigned char *dest, in
         for (int j = 0; j < 8; j++) {
             int idxComponentY = (chunkStartY * rowsPerChunk + rowsSoFar) * 8 + j;
 
-            for (int k = 0; k < 8; k++) {
+            for (int k = 0; k < 8; k += 2) {
                 int idxComponentX = (chunkStartX * colsPerChunk + tilesSoFar) * 8 + k;
                 unsigned char srcPixelPair = *src++;
                 unsigned char leftPixel = srcPixelPair & 0xF;
@@ -139,7 +139,6 @@ static void Convert8BppFrom4BppTiles(unsigned char *src, unsigned char *dest, in
 
                 dest[idxComponentY * pitch + idxComponentX] = ((palIndex - 1) << 4) | leftPixel;
                 dest[idxComponentY * pitch + idxComponentX + 1] = ((palIndex - 1) << 4) | rightPixel;
-                k++;
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -126,7 +126,7 @@ static void Convert8BppFrom4BppTiles(unsigned char src, unsigned char dest, in`
`126`	`126`	`for (int j = 0; j < 8; j++) {`
`127`	`127`	`int idxComponentY = (chunkStartY * rowsPerChunk + rowsSoFar) * 8 + j;`
`128`	`128`
`129`		`- for (int k = 0; k < 8; k++) {`
	`129`	`+ for (int k = 0; k < 8; k += 2) {`
`130`	`130`	`int idxComponentX = (chunkStartX * colsPerChunk + tilesSoFar) * 8 + k;`
`131`	`131`	`unsigned char srcPixelPair = *src++;`
`132`	`132`	`unsigned char leftPixel = srcPixelPair & 0xF;`
`@@ -139,7 +139,6 @@ static void Convert8BppFrom4BppTiles(unsigned char src, unsigned char dest, in`
`139`	`139`
`140`	`140`	`dest[idxComponentY * pitch + idxComponentX] = ((palIndex - 1) << 4) \| leftPixel;`
`141`	`141`	`dest[idxComponentY * pitch + idxComponentX + 1] = ((palIndex - 1) << 4) \| rightPixel;`
`142`		`- k++;`
`143`	`142`	`}`
`144`	`143`	`}`
`145`	`144`