Let's use enums for groupnumber

src/display3d.cpp

@@ -3622,15 +3622,20 @@ static void drawDroidAndStructureSelections()
 /// X/Y offset to display the group number at
 #define GN_X_OFFSET	(8)
 #define GN_Y_OFFSET	(3)
+enum GROUPNUMBER_TYPE
+{
+	GN_NORMAL,
+	GN_DAMAGED,
+	GN_FACTORY
+};
 /// rendering of the unit's group next to the unit itself, 
 /// or the group that will be assigned to the unit after production in the factory
 static void	drawGroupNumber(BASE_OBJECT *psObject)
 {
 	UWORD id = UWORD_MAX;
 	UBYTE groupNumber = UBYTE_MAX;
 	int32_t x = 0, y = 0;
-	bool isFactory = false;
-	bool wentToRepair = false;
+	GROUPNUMBER_TYPE groupNumberType = GN_NORMAL;
 
 	if (auto *psDroid = dynamic_cast<DROID*>(psObject))
 	{
@@ -3642,7 +3647,7 @@ static void	drawGroupNumber(BASE_OBJECT *psObject)
 
 		if (psDroid->repairGroup != UBYTE_MAX) {
 			groupNumber = psDroid->repairGroup;
-			wentToRepair = true;
+			groupNumberType = GN_DAMAGED;
 		} else {
 			groupNumber = psDroid->group;
 		}
@@ -3660,7 +3665,7 @@ static void	drawGroupNumber(BASE_OBJECT *psObject)
 		y = scrY - GN_Y_OFFSET;
 
 		groupNumber = psStruct->productToGroup;
-		isFactory = true;
+		groupNumberType = GN_FACTORY;
 	}
 
 	switch (groupNumber)
@@ -3701,17 +3706,19 @@ static void	drawGroupNumber(BASE_OBJECT *psObject)
 
 	if (id != UWORD_MAX)
 	{
-		if (wentToRepair)
+		switch (groupNumberType)
 		{
+		case GN_NORMAL:
+			iV_DrawImage(IntImages, id, x, y);
+			break;
+		case GN_DAMAGED:
 			iV_DrawImageTint(IntImages, id, x, y, pal_RGBA(255, 0, 0, 255) /* red */);
-		}
-		else if (isFactory)
-		{
+			break;
+		case GN_FACTORY:
 			iV_DrawImageTint(IntImages, id, x, y, pal_RGBA(255, 220, 115, 255) /* gold */);
-		}
-		else
-		{
-			iV_DrawImage(IntImages, id, x, y);
+			break;
+		default:
+			break;
 		}
 	}
 }