1. Semantic Issue enumeration values uiimage orientation not handled in switch. here is the method it is referring to:

switch (self.imageOrientation) {
case UIImageOrientationDown: // EXIF = 3
case UIImageOrientationDownMirrored: // EXIF = 4
transform = CGAffineTransformTranslate(transform, newSize.width, newSize.height);
transform = CGAffineTransformRotate(transform, M_PI);
break;

2. symantec issue 4 enumeration values not handled in switch uiimage orientation up, uiimage orientaton down, uiimage orientation left, uiimage orientation right. Here is the method:

switch (self.imageOrientation) {
case UIImageOrientationUpMirrored: // EXIF = 2
case UIImageOrientationDownMirrored: // EXIF = 4
transform = CGAffineTransformTranslate(transform, newSize.width, 0);
transform = CGAffineTransformScale(transform, -1, 1);
break;

I want to take images taken with the iphone camera (landscape or portrait) and resize them to a manageable size (around 120kb) while preserving most of the quality.

What should I send as the newSize for the resizedImage? 640×480 or 480×640 ?

I didn’t understand the “result will always be “up”.
it means the resized image will always result in 480×640 ?

Thanks alot!

This is also missing the one thing I need which is scale an image to a (single) max width/height, preserving the aspect ratio. Why should thumbnails be square, that makes no sense to me. If I am displaying square thumbnails, it’s much better to handle this on the UIImageView level. Let the view decide whether to display the image as aspect fit, fill, etc rather than putting this in the scaled image.

When you think about it, who cares so much about the input file because we are defining here what the output image will be. So, in UIImage+Resize.m around line 120, replace the line CGContextRef bitmap = … with

CGContextRef bitmap = CGBitmapContextCreate(
NULL,
newRect.size.width,
newRect.size.height,
8, /* bits per channel */
(newRect.size.width * 4), /* 4 channels per pixel * numPixels/row */
CGColorSpaceCreateDeviceRGB(),
kCGImageAlphaPremultipliedLast
);

and that seemed to make the errors go away. Images look ok too.

Change the resizeImage method in UIImage+Resize.m to directly specify rather than derive the colour space and alpha information.

// make the following changes.

CGColorSpaceRef colorSpace = CGColorSpaceCreateDeviceRGB();

CGContextRef bitmap = CGBitmapContextCreate(NULL, newRect.size.width, newRect.size.height, 8, 4 * newRect.size.width, colorSpace, kCGImageAlphaPremultipliedFirst);

CGColorSpaceRelease(colorSpace);

// rest of code continues.

Cheers,
Matt