Quel est le bon modèle de gestion de mémoire pour buffer-> CGImageRef-> UIImage?

Question

J'ai une fonction qui prend des données bitmap et renvoie un UIImage *. Il ressemble à quelque chose comme:

UIImage * makeAnImage() 
{ 
    unsigned char * pixels = malloc(...); 
    // ... 
    CGDataProviderRef provider = CGDataProviderCreateWithData(NULL, pixels, pixelBufferSize, NULL); 
    CGImageRef imageRef = CGImageCreate(..., provider, ...); 
    UIImage * image = [[UIImage alloc] initWithCGImage:imageRef]; 
    return [image autorelease]; 
} 

Quelqu'un peut-il expliquer exactement qui possède quelle mémoire ici? Je veux nettoyer correctement, mais je ne suis pas sûr de savoir comment le faire en toute sécurité. Les documents sont flous sur ceux-ci. Si je free pixels à la fin de cette fonction après avoir créé l'UIImage, puis utiliser le UIImage, je plante. Si je libère le fournisseur ou l'imageRef après avoir créé l'UIImage, je ne vois pas de crash, mais ils passent apparemment les pixels complètement, donc je suis nerveux à propos de la libération de ces états intermédiaires.

(Je sais par CF docs que je devrais avoir besoin d'appeler release sur ces deux derniers parce qu'ils proviennent des fonctions Create, mais puis-je le faire avant l'utilisation de UIImage?) Je peux probablement utiliser le callloc callback du fournisseur nettoyer le tampon de pixels, mais quoi d'autre?

Merci!

Answer 1

La règle du pouce ici est "-release * si vous n'en avez pas besoin".

Parce que vous ne avez plus besoin provider et imageRef après, vous devriez -release tous, à savoir

UIImage * image = [[UIImage alloc] initWithCGImage:imageRef]; 
CGDataProviderRelease(provider); 
CGImageRelease(imageRef); 
return [image autorelease]; 

pixel est pas géré par ref-comptage, de sorte que vous devez dire à l'API CG pour les libérer pour vous quand nécessaire. Pour ce faire:

void releasePixels(void *info, const void *data, size_t size) { 
    free((void*)data); 
} 
.... 

CGDataProviderRef provider = CGDataProviderCreateWithData(NULL, pixels, pixelBufferSize, releasePixels); 

Par ailleurs, vous pouvez utiliser +imageWithCGImage: au lieu de [[[* alloc] initWithCGImage:] autorelease]. Encore mieux, il ya +imageWithData: donc vous n'avez pas besoin de jouer avec le CG et malloc choses.

(*. Sauf lorsque le retainCount est déjà soi-disant zéro depuis le début)

Answer 2

Oui, ce code me rend mal à l'aise. En tant qu'ancien rule-to-live, j'essaie de ne pas mélanger et de faire correspondre C et C++, et C/Objective-C dans la même fonction/méthode/sélecteur.

Que diriez-vous de diviser cela en deux méthodes. Avoir changé cette makeAnImage en makeAnImageRef et remonter la création UIImage dans un autre sélecteur Obj-C.

Answer 3

unsigned char * pixels = malloc(...); 

Vous possédez le tampon pixels parce que vous mallocked il.

CGDataProviderRef provider = CGDataProviderCreateWithData(NULL, pixels, pixelBufferSize, NULL); 

Core Graphics suit les règles de base de la Fondation. Vous possédez le fournisseur de données parce que vous Created il.

Vous n'avez pas fourni de rappel de version, vous disposez donc toujours du tampon pixels. Si vous aviez fourni un rappel de version, l'objet CGDataProvider prendrait possession du tampon ici. (Généralement une bonne idée.)

CGImageRef imageRef = CGImageCreate(..., provider, ...); 

Vous possédez l'objet CGImage car vous l'avez créé.

UIImage * image = [[UIImage alloc] initWithCGImage:imageRef]; 

Vous possédez l'objet UIImage car vous l'avez alloué.

Vous possédez également l'objet CGImage. Si l'objet UIImage veut posséder l'objet CGImage, il le conservera ou fera sa propre copie.

return [image autorelease]; 

Vous abandonnez votre propriété de l'image. Donc, votre code fuit les pixels (vous n'avez pas transféré la propriété au fournisseur de données et vous ne les avez pas libérés vous-même), le fournisseur de données (vous ne l'avez pas publié) et le CGImage (vous n'avez pas t le relâcher). Une version fixe transfèrerait la propriété des pixels au fournisseur de données, et libérerait à la fois le fournisseur de données et la CGImage au moment où l'UIImage est prêt. Ou, utilisez simplement imageWithData:, comme le suggère KennyTM.

Answer 4

unsigned char * pixels = malloc(...); 

J'ai aussi eu problème avec malloc/libre après avoir utilisé CGImageCreate j'ai finalement trouvé une solution simple et bonne. Je viens de remplacer la ligne:

CGDataProviderRef provider = CGDataProviderCreateWithData(NULL, pixels, pixelBufferSize, NULL); 

avec:

NSData *data = [NSData dataWithBytes:pixels length:pixelBufferSize]; 
CGDataProviderRef provider = CGDataProviderCreateWithCFData((CFDataRef)data); 

Juste après que je pouvais libérer de la mémoire mallocked:

free (pixels);