# HG changeset patch
# User Charles Pigott <charlespigott@googlemail.com>
# Date 2020-01-05 23:05:28
# Node ID c84c014df40067b0b0b2c76ecc8da7c936f80a54
# Parent  b9909505e53018aa7ea796771988de5955ba7335

Fix e558aa8: Compiler warning about unused value (and move some variable declarations to where they're used)

diff --git a/src/screenshot.cpp b/src/screenshot.cpp
--- a/src/screenshot.cpp
+++ b/src/screenshot.cpp
@@ -893,12 +893,8 @@ static Owner GetMinimapOwner(TileIndex t
 
 static void MinimapScreenCallback(void *userdata, void *buf, uint y, uint pitch, uint n)
 {
-	uint32 *ubuf;
-	uint num, row, col;
-	byte val;
+	/* Fill with the company colours */
 	byte owner_colours[OWNER_END + 1];
-
-	/* Fill with the company colours */
 	for (const Company *c : Company::Iterate()) {
 		owner_colours[c->index] = MKCOLOUR(_colour_gradient[c->colour][5]);
 	}
@@ -910,15 +906,15 @@ static void MinimapScreenCallback(void *
 	owner_colours[OWNER_DEITY]   = PC_DARK_GREY; // industry
 	owner_colours[OWNER_END]     = PC_BLACK;
 
-	ubuf = (uint32 *)buf;
-	num = (pitch * n);
+	uint32 *ubuf = (uint32 *)buf;
+	uint num = (pitch * n);
 	for (uint i = 0; i < num; i++) {
-		row = y + (int)(i / pitch);
-		col = (MapSizeX() - 1) - (i % pitch);
+		uint row = y + (int)(i / pitch);
+		uint col = (MapSizeX() - 1) - (i % pitch);
 
 		TileIndex tile = TileXY(col, row);
 		Owner o = GetMinimapOwner(tile);
-		val = owner_colours[o];
+		byte val = owner_colours[o];
 
 		uint32 colour_buf = 0;
 		colour_buf  = (_cur_palette.palette[val].b << 0);
@@ -926,7 +922,7 @@ static void MinimapScreenCallback(void *
 		colour_buf |= (_cur_palette.palette[val].r << 16);
 
 		*ubuf = colour_buf;
-		*ubuf++;   // Skip alpha
+		ubuf++;   // Skip alpha
 	}
 }