Changeset 1315

Show
Ignore:
Timestamp:
06/29/09 14:37:31 (8 months ago)
Author:
tsnorri
Message:

Changed calls to CFRetain and CFRelease as per Apple's recommendation
- Fixes #209.

Location:
trunk/Sources
Files:
5 modified

Legend:

Unmodified
Added
Removed

  • trunk/Sources/BXDatabaseContext.m

    r1289 r1315  
    28882888    NSMutableArray* retval = nil; 
    28892889     
    2890     NSMutableData* attributeBuffer = (id) CFRetain ([NSMutableData data]); 
     2890    NSMutableData* attributeBuffer = [NSMutableData data]; 
    28912891    AddKeychainAttributeString (kSecAccountItemAttr, [mDatabaseURI user], attributeBuffer); 
    28922892    AddKeychainAttributeString (kSecServerItemAttr,  [mDatabaseURI host], attributeBuffer); 
    … …  
    29272927         
    29282928        //For GC. 
    2929         CFRelease (attributeBuffer); 
     2929        [attributeBuffer self]; 
    29302930         
    29312931    return retval; 

  • trunk/Sources/BXLogger.m

    r1230 r1315  
    121121        const char* file = LastPathComponent (fileName); 
    122122         
    123         NSString* date = (id) CFRetain ([[NSDate date] descriptionWithCalendarFormat: @"%Y-%m-%d %H:%M:%S.%F" timeZone: nil locale: nil]); 
    124         NSString* message = (id) CFRetain ([[[NSString alloc] initWithFormat: messageFmt arguments: args] autorelease]); 
     123        NSString* date = [[NSDate date] descriptionWithCalendarFormat: @"%Y-%m-%d %H:%M:%S.%F" timeZone: nil locale: nil]; 
     124        NSString* message = [[[NSString alloc] initWithFormat: messageFmt arguments: args] autorelease]; 
    125125                 
    126126        const char isMain = ([NSThread isMainThread] ? 'm' : 's'); 
    … …  
    129129         
    130130        //For GC. 
    131         CFRelease (date); 
    132         CFRelease (message); 
     131        [date self]; 
     132        [message self]; 
    133133         
    134134        if (executable) 

  • trunk/Sources/BXPGSQLScriptReader.m

    r1188 r1315  
    6767        } 
    6868         
    69         NSString* pathString = (id) CFRetain ([fileURL path]); 
     69        NSString* pathString = [fileURL path]; 
    7070        const char* path = [pathString UTF8String]; 
    7171        int fd = open (path, O_RDONLY | O_SHLOCK); 
    … …  
    8080         
    8181        //For GC. 
    82         CFRelease (pathString); 
     82        [pathString self]; 
    8383        return retval; 
    8484} 

  • trunk/Sources/PGTSConnector.m

    r1257 r1315  
    4646CopyConnectionString (NSDictionary* connectionDict) 
    4747{ 
    48         //-UTF8String assigns an internal pointer, hence the CFRetain + CFRelease. 
    4948        NSMutableString* connectionString = [NSMutableString string]; 
    50         CFRetain (connectionString); 
    5149        NSEnumerator* e = [connectionDict keyEnumerator]; 
    5250        NSString* currentKey; 
    … …  
    5856        } 
    5957        char* retval = strdup ([connectionString UTF8String]); 
    60         CFRelease (connectionString); 
     58 
     59        //For GC. 
     60        [connectionString self]; 
    6161        return retval; 
    6262} 

  • trunk/Sources/PGTSQuery.m

    r1277 r1315  
    150150{ 
    151151        NSMutableString* desc = [NSMutableString string]; 
    152         CFRetain (desc); 
    153152        for (int i = 0; i < nParams; i++) 
    154153        { 
    … …  
    162161        } 
    163162        char* retval = strdup ([desc UTF8String] ?: ""); 
    164         CFRelease (desc); 
     163         
     164        //For GC. 
     165        [desc self]; 
    165166        return retval; 
    166167} 
    … …  
    175176                NSUInteger nParams = [self parameterCount]; 
    176177                NSArray* parameterObjects = [[mParameters PGTSCollect] PGTSParameter: connection]; 
    177                 if (parameterObjects) CFRetain (parameterObjects); 
    178178                 
    179179                const char** paramValues  = calloc (nParams, sizeof (char *)); 
    … …  
    217217                        [(id) [connection delegate] PGTSConnection: connection sentQuery: self]; 
    218218                 
    219                 if (parameterObjects) CFRelease (parameterObjects); 
    220219                free (paramTypes); 
    221220                free (paramValues); 
    222221                free (paramLengths); 
    223222                free (paramFormats); 
     223 
     224                //For GC. 
     225                [parameterObjects self]; 
    224226        } 
    225227    return retval;