Skip to main content

Is a "CFRelease” needed if I am using CGImageSourceCreateWithData from a (__bridge)?



My question is very simple, I would like to know if the method "CGImageSourceCreateWithData" creates a new object copying the data I provide so that that I have to release it when I don't need it anymore, or if it just creates a reference to the data I already have so that if I release it I will lose this data (and possibly have bad access errors).





The issue is related to using (__bridge CFDataRef) as the source data. Which let's me use Core Foundation objects as toll-free in ARC mode.





Consider the following function (or method, not sure how it's called):







- (void)saveImageWithData:(NSData*)jpeg andDictionary:(NSDictionary*)dicRef andName:(NSString*)name

{

[self setCapturedImageName:name];



CGImageSourceRef source ;



// Notice here how I use __bridge

source = CGImageSourceCreateWithData((__bridge CFDataRef)jpeg, NULL);



CFStringRef UTI = CGImageSourceGetType(source);



NSMutableData *dest_data = [NSMutableData data];



// And here I use it again

CGImageDestinationRef destination = CGImageDestinationCreateWithData((__bridge CFMutableDataRef)dest_data,UTI,1,NULL);



CGImageDestinationAddImageFromSource(destination,source,0, (__bridge CFDictionaryRef) dicRef);



BOOL success = NO;

success = CGImageDestinationFinalize(destination);



if(!success) {

NSLog(@"***Could not create data from image destination ***");

}



// This only saves to the disk

NSArray *paths = NSSearchPathForDirectoriesInDomains(NSDocumentDirectory, NSUserDomainMask, YES);

NSString *documentsDirectory = [paths objectAtIndex:0]; // Get documents folder

NSString *dataPath = [documentsDirectory stringByAppendingPathComponent:@"ARPictures"];



NSError *error;

if (![[NSFileManager defaultManager] fileExistsAtPath:dataPath])

[[NSFileManager defaultManager] createDirectoryAtPath:dataPath withIntermediateDirectories:NO attributes:nil error:&error]; //Create folder



NSString *fullPath = [dataPath stringByAppendingPathComponent:[NSString stringWithFormat:@"%@.jpg", name]]; //add our image to the path



[dest_data writeToFile:fullPath atomically:YES];





self.img = [[UIImage alloc] initWithData:dest_data];

self.capturedImageData = [[NSData alloc] initWithData:dest_data];



//This is what im not sure if i should use

CFRelease(destination);

CFRelease(source);



}







My concern is about memory leaks or dealocating things I should not.





Thanks


Comments

  1. You are doing it correctly. It does not actually matter whether these routines make copies or not. What matters is that you CFRelease what you "Create" (or "Copy"). Everything here looks correct. __bridge is appropriate when passing parameters because you're not actually transferring the object from CF to Cocoa or vice versa. You're just temporarily "bridging" (casting) it.

    ReplyDelete

Post a Comment

Popular posts from this blog

Why is this Javascript much *slower* than its jQuery equivalent?

I have a HTML list of about 500 items and a "filter" box above it. I started by using jQuery to filter the list when I typed a letter (timing code added later): $('#filter').keyup( function() { var jqStart = (new Date).getTime(); var search = $(this).val().toLowerCase(); var $list = $('ul.ablist > li'); $list.each( function() { if ( $(this).text().toLowerCase().indexOf(search) === -1 ) $(this).hide(); else $(this).show(); } ); console.log('Time: ' + ((new Date).getTime() - jqStart)); } ); However, there was a couple of seconds delay after typing each letter (particularly the first letter). So I thought it may be slightly quicker if I used plain Javascript (I read recently that jQuery's each function is particularly slow). Here's my JS equivalent: document.getElementById('filter').addEventListener( 'keyup', function () { var jsStart = (new Date).getTime()...

Is it possible to have IF statement in an Echo statement in PHP

Thanks in advance. I did look at the other questions/answers that were similar and didn't find exactly what I was looking for. I'm trying to do this, am I on the right path? echo " <div id='tabs-".$match."'> <textarea id='".$match."' name='".$match."'>". if ($COLUMN_NAME === $match) { echo $FIELD_WITH_COLUMN_NAME; } else { } ."</textarea> <script type='text/javascript'> CKEDITOR.replace( '".$match."' ); </script> </div>"; I am getting the following error message in the browser: Parse error: syntax error, unexpected T_IF Please let me know if this is the right way to go about nesting an IF statement inside an echo. Thank you.