UnrealXR/raylib
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

REDESIGNED: ImageFlipHorizontal(), optimized #1218

Browse source 
Added several optimized options for future reference (if required)
This commit is contained in:
raysan5 2020-06-07 11:52:21 +02:00
parent 93d6dd3023
commit d278eae4a3
1 changed files with 35 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

51
src/textures.c
View file

@ -1605,26 +1605,45 @@ void ImageFlipHorizontal(Image *image)
// Security check to avoid program crash // Security check to avoid program crash
if ((image->data == NULL) || (image->width == 0) || (image->height == 0)) return; if ((image->data == NULL) || (image->width == 0) || (image->height == 0)) return;
Color *srcPixels = GetImageData(*image); if (image->mipmaps > 1) TRACELOG(LOG_WARNING, "Image manipulation only applied to base mipmap level");
Color *dstPixels = (Color *)RL_MALLOC(image->width*image->height*sizeof(Color)); if (image->format >= COMPRESSED_DXT1_RGB) TRACELOG(LOG_WARNING, "Image manipulation not supported for compressed formats");
else
for (int y = 0; y < image->height; y++)
{ {
for (int x = 0; x < image->width; x++) int dataSize = GetPixelDataSize(image->width, image->height, image->format);
int bytesPerPixel = dataSize/(image->width*image->height);
unsigned char *flippedData = (unsigned char *)RL_MALLOC(dataSize);
for (int y = 0; y < image->height; y++)
{ {
dstPixels[y*image->width + x] = srcPixels[y*image->width + (image->width - 1 - x)]; for (int x = 0; x < image->width; x++)
{
// OPTION 1: Move pixels with memcopy()
//memcpy(flippedData + (y*image->width + x)*bytesPerPixel, ((unsigned char *)image->data) + (y*image->width + (image->width - 1 - x))*bytesPerPixel, bytesPerPixel);
// OPTION 2: Just copy data pixel by pixel
for (int i = 0; i < bytesPerPixel; i++) flippedData[(y*image->width + x)*bytesPerPixel + i] = ((unsigned char *)image->data)[(y*image->width + (image->width - 1 - x))*bytesPerPixel + i];
}
} }
RL_FREE(image->data);
image->data = flippedData;
/*
// OPTION 3: Faster implementation (specific for 32bit pixels)
// NOTE: It does not require additional allocations
uint32_t *ptr = (uint32_t *)image->data;
for (int y = 0; y < image->height; y++)
{
for (int x = 0; x < image->width/2; x++)
{
uint32_t backup = ptr[y*image->width + x];
ptr[y*image->width + x] = ptr[y*image->width + (image->width - 1 - x)];
ptr[y*image->width + (image->width - 1 - x)] = backup;
}
}
*/
} }
int format = image->format;
RL_FREE(image->data);
image->data = dstPixels;
image->format = UNCOMPRESSED_R8G8B8A8;
ImageFormat(image, format);
RL_FREE(srcPixels);
} }
// Rotate image clockwise 90deg // Rotate image clockwise 90deg