skip unecessary format conversion on load

Author: HeliumAnt

Source/Managers/ActivityMan.cpp

@@ -206,6 +206,7 @@ bool ActivityMan::SaveCurrentGame(const std::string& fileName) {
 
 			              SDL_Palette* palette = ContentFile::DefaultPaletteToSDL();
 			              SDL_SetSurfacePalette(image, palette);
+						  SDL_SetSurfaceColorKey(image, false, 0);
 
 			              bool result = IMG_SavePNG_IO(image, stream, false);
 			              SDL_FlushIO(stream);
@@ -310,11 +311,16 @@ bool ActivityMan::LoadAndLaunchGame(const std::string& fileName) {
 		SDL_Surface* image = stream ? IMG_LoadPNG_IO(stream) : nullptr;
 		SDL_CloseIO(stream);
 
+		int bitDepth = SDL_GetPixelFormatDetails(image->format)->bits_per_pixel;
 		SDL_Palette* palette = ContentFile::DefaultPaletteToSDL();
-		SDL_Surface* newImage = SDL_ConvertSurfaceAndColorspace(image, SDL_PIXELFORMAT_INDEX8, palette, SDL_COLORSPACE_UNKNOWN, 0);
+		if (bitDepth != 8) {
+			SDL_Surface* newImage = SDL_ConvertSurfaceAndColorspace(image, SDL_PIXELFORMAT_INDEX8, palette, SDL_COLORSPACE_UNKNOWN, 0);
+			SDL_DestroySurface(image);
+			image = newImage;
+		} else {
+			SDL_SetSurfacePalette(image, palette);
+		}
 		SDL_DestroyPalette(palette);
-		SDL_DestroySurface(image);
-		image = newImage;
 
 		free(buffer);
 		return image;