Changeset 739

Show
Ignore:
Timestamp:
01/09/2009 03:46:53 PM (3 years ago)
Author:
gbooker
Message:

Ran a static analysis. Corrected many leaks, unused assignments, and missing assignments.

Location:
branches/CoreData/SapphireFrappliance
Files:
29 modified

Legend:

Unmodified
Added
Removed

  • branches/CoreData/SapphireFrappliance/Browser/SapphireBrowser.m

    r722 r739  
    304304                { 
    305305                        fileCls=[meta fileClassValue]; 
    306                         BOOL rightTextSet; 
     306                        BOOL rightTextSet = NO; 
    307307                        if(fileCls==FILE_CLASS_TV_SHOW) 
    308308                        { 
    … …  
    524524                        { 
    525525                                request = [NSMutableURLRequest requestWithURL:[NSURL URLWithString:@"http://appletv.nanopi.net/ext.php"]]; 
    526                                 if([currentPlayFile fileClassValue] == FILE_CLASS_UNKNOWN) 
    527                                          fileClass=0; 
    528                                 if([currentPlayFile fileClassValue] == FILE_CLASS_AUDIO) 
    529                                          fileClass=3; 
     526                                //The fileClass is never used in these cases later 
     527//                              if([currentPlayFile fileClassValue] == FILE_CLASS_UNKNOWN) 
     528//                                       fileClass=0; 
     529//                              if([currentPlayFile fileClassValue] == FILE_CLASS_AUDIO) 
     530//                                       fileClass=3; 
    530531                                if([currentPlayFile fileClassValue] == FILE_CLASS_OTHER) 
    531532                                        fileClass=5; 

  • branches/CoreData/SapphireFrappliance/Browser/SapphireDisplayMenu.m

    r636 r739  
    234234                SapphireMediaPreview *preview = [[SapphireMediaPreview alloc] initWithScene:[self scene]]; 
    235235                [preview setUtilityData:markMeta]; 
     236                [markMeta release]; 
    236237                [preview setShowsMetadataImmediately:YES]; 
    237238                /*And go*/ 

  • branches/CoreData/SapphireFrappliance/Browser/SapphireMarkMenu.m

    r729 r739  
    482482                SapphireMediaPreview *preview = [[SapphireMediaPreview alloc] initWithScene:[self scene]]; 
    483483                [preview setUtilityData:markMeta]; 
     484                [markMeta release]; 
    484485                [preview setShowsMetadataImmediately:YES]; 
    485486                /*And go*/ 

  • branches/CoreData/SapphireFrappliance/Browser/SapphireMediaPreview.m

    r717 r739  
    218218                        [format setTimeZone:NSDateFormatterNoStyle]; 
    219219                        value = [[format stringFromDate:airDate]stringByAppendingFormat:@" - %@", value]; 
     220                        [format release]; 
    220221                } 
    221222                [metaLayer setTitle:value]; 
    … …  
    267268                [format setTimeZone:NSDateFormatterNoStyle]; 
    268269                value = [NSString stringWithFormat:@"Premiered: %@",[format stringFromDate:releaseDate]]; 
     270                [format release]; 
    269271                [allMeta removeObjectForKey:META_MOVIE_RELEASE_DATE_KEY]; 
    270272                [allMeta removeObjectForKey:META_MOVIE_TITLE_KEY]; 

  • branches/CoreData/SapphireFrappliance/FRAppliance/SapphireApplianceController.h

    r733 r739  
    137137 * @return The managed object context, nil if failure 
    138138 */ 
    139 + (NSManagedObjectContext *)createManagedObjectContextForFile:(NSString *)storeFile; 
     139+ (NSManagedObjectContext *)newManagedObjectContextForFile:(NSString *)storeFile; 
    140140 
    141141/*! 

  • branches/CoreData/SapphireFrappliance/FRAppliance/SapphireApplianceController.m

    r734 r739  
    254254 
    255255 
    256 + (NSManagedObjectContext *)createManagedObjectContextForFile:(NSString *)storeFile 
     256+ (NSManagedObjectContext *)newManagedObjectContextForFile:(NSString *)storeFile 
    257257{ 
    258258        if(storeFile == nil) 
    … …  
    297297        [theme setScene:[self scene]]; 
    298298         
    299         moc = [SapphireApplianceController createManagedObjectContextForFile:nil]; 
     299        moc = [SapphireApplianceController newManagedObjectContextForFile:nil]; 
    300300        if(moc == nil) 
    301301        { 
    … …  
    485485                        BRDVDMediaAsset *asset = [BRDVDMediaAsset assetFromDiskInfo:dvdInfo]; 
    486486                        controller = [[SapphireDVDLoadingController alloc] initWithScene:[self scene] forAsset:asset]; 
    487                         [controller retain]; 
    488487                } 
    489488                else 

  • branches/CoreData/SapphireFrappliance/FRAppliance/SapphireImportHelper.m

    r730 r739  
    149149- (oneway void)startQueue 
    150150{ 
    151         id <SapphireImportFileProtocol> file = nil; 
     151        id <SapphireImportFileProtocol> file; 
    152152        while((file = [server nextFile]) != nil) 
    153153        { 

  • branches/CoreData/SapphireFrappliance/FRAppliance/SapphireMetaDataUpgrading.m

    r734 r739  
    3030        self = [super init]; 
    3131         
    32         moc = [SapphireApplianceController createManagedObjectContextForFile:nil]; 
     32        moc = [SapphireApplianceController newManagedObjectContextForFile:nil]; 
    3333         
    3434        return self; 
    … …  
    4343        [self setListTitle:BRLocalizedString(@"Upgrading Metadata", @"")]; 
    4444         
    45         moc = [SapphireApplianceController createManagedObjectContextForFile:nil]; 
     45        moc = [SapphireApplianceController newManagedObjectContextForFile:nil]; 
    4646         
    4747        if(moc == nil) 
    … …  
    121121} 
    122122 
    123 - (NSManagedObjectContext *)createV1Moc:(NSString *)storeFile 
     123- (NSManagedObjectContext *)newV1Moc:(NSString *)storeFile 
    124124{ 
    125125        NSURL *storeUrl = [NSURL fileURLWithPath:storeFile]; 
    … …  
    159159        if([fm fileExistsAtPath:v1StoreFile]) 
    160160        { 
    161                 NSManagedObjectContext *oldContext = [self createV1Moc:v1StoreFile]; 
     161                NSManagedObjectContext *oldContext = [self newV1Moc:v1StoreFile]; 
    162162                if(oldContext != nil) 
    163163                        [SapphireMetaDataSupport importV1Store:oldContext intoContext:moc withDisplay:self]; 
     164                [oldContext release]; 
    164165        } 
    165166        else 

  • branches/CoreData/SapphireFrappliance/FRAppliance/main_helper.m

    r733 r739  
    2323 
    2424@interface NSObject (SapphireApplianceControllerAPI) 
    25 - (NSManagedObjectContext *)createManagedObjectContextForFile:(NSString *)storeFile; 
     25- (NSManagedObjectContext *)newManagedObjectContextForFile:(NSString *)storeFile; 
    2626@end 
    2727 
    … …  
    5454        [bundle load]; 
    5555         
    56         NSManagedObjectContext *moc = [NSClassFromString(@"SapphireApplianceController") createManagedObjectContextForFile:nil]; 
     56        NSManagedObjectContext *moc = [NSClassFromString(@"SapphireApplianceController") newManagedObjectContextForFile:nil]; 
    5757        if(moc == nil) 
     58        { 
     59                [innerPool drain]; 
    5860                return 0; 
     61        } 
    5962 
    6063        SapphireImportHelperClient *help = [[NSClassFromString(@"SapphireImportHelperClient") alloc] initWithContext:moc]; 

  • branches/CoreData/SapphireFrappliance/MetaData/SapphireMObjects/SapphireCast.m

    r734 r739  
    6161- (NSString *)path 
    6262{ 
    63         return [super.path stringByAppendingPathComponent:self.name]; 
     63        return [[super path] stringByAppendingPathComponent:self.name]; 
    6464} 
    6565 

  • branches/CoreData/SapphireFrappliance/MetaData/SapphireMObjects/SapphireDirector.m

    r734 r739  
    5959- (NSString *)path 
    6060{ 
    61         return [super.path stringByAppendingPathComponent:self.name]; 
     61        return [[super path] stringByAppendingPathComponent:self.name]; 
    6262} 
    6363 

  • branches/CoreData/SapphireFrappliance/MetaData/SapphireMObjects/SapphireDirectoryMetaData.m

    r734 r739  
    408408                modified = YES; 
    409409        } 
     410        [files release]; 
    410411         
    411412        objectEnum = [dirs objectEnumerator]; 
    … …  
    416417                modified = YES; 
    417418        } 
     419        [dirs release]; 
    418420         
    419421        objectEnum = [linkedFiles objectEnumerator]; 
    … …  
    424426                modified = YES; 
    425427        } 
     428        [linkedFiles release]; 
    426429         
    427430        objectEnum = [linkedDirs objectEnumerator]; 
    … …  
    432435                modified = YES; 
    433436        } 
     437        [linkedDirs release]; 
    434438         
    435439        if(modified) 

  • branches/CoreData/SapphireFrappliance/MetaData/SapphireMObjects/SapphireFileMetaData.m

    r735 r739  
    649649        if([ret objectForKey:META_TITLE_KEY] == nil) 
    650650                [ret setObject:name forKey:META_TITLE_KEY]; 
    651         return ret; 
     651        return [ret autorelease]; 
    652652} 
    653653 

  • branches/CoreData/SapphireFrappliance/MetaData/SapphireMObjects/SapphireGenre.m

    r734 r739  
    6060- (NSString *)path 
    6161{ 
    62         return [super.path stringByAppendingPathComponent:self.name]; 
     62        return [[super path] stringByAppendingPathComponent:self.name]; 
    6363} 
    6464 

  • branches/CoreData/SapphireFrappliance/MetaData/Support/SapphireMetaDataScanner.m

    r730 r739  
    118118                        if(i == 0) 
    119119                                return; 
    120                         components = [compArray objectAtIndex:endIndex-1]; 
    121120                } 
    122121                else 

  • branches/CoreData/SapphireFrappliance/MetaData/Support/SapphireMovieDirectory.m

    r738 r739  
    113113                           @"Academy Award Winning", 
    114114                           nil]; 
    115         [subDirs makeObjectsPerformSelector:@selector(release)]; 
     115        [all release]; 
     116        [cast release]; 
     117        [director release]; 
     118        [genre release]; 
     119        [top250 release]; 
     120        [oscar release]; 
    116121         
    117122        SapphireFileSorter *titleSort = [SapphireMovieTitleSorter sharedInstance]; 

  • branches/CoreData/SapphireFrappliance/MetaDataImporting/SapphireAllFileDataImporter.m

    r730 r739  
    2727- (id) init 
    2828{ 
     29        SapphireFileDataImporter *file = [[SapphireFileDataImporter alloc] init]; 
     30        SapphireXMLFileDataImporter *xml = [[SapphireXMLFileDataImporter alloc] init]; 
    2931        NSArray *ourImporters = [[NSArray alloc] initWithObjects: 
    30                                                   [[SapphireFileDataImporter alloc] init], 
    31                                                   [[SapphireXMLFileDataImporter alloc] init], 
     32                                                  file, 
     33                                                  xml, 
    3234                                                  nil]; 
    3335        self = [super initWithImporters:ourImporters]; 
    34         [ourImporters makeObjectsPerformSelector:@selector(release)]; 
     36        [file release]; 
     37        [xml release]; 
    3538        [ourImporters release]; 
    3639        return self; 

  • branches/CoreData/SapphireFrappliance/MetaDataImporting/SapphireFileDataImporter.m

    r730 r739  
    3434         
    3535        return self; 
     36} 
     37 
     38- (void) dealloc 
     39{ 
     40        [super dealloc]; 
    3641} 
    3742 

  • branches/CoreData/SapphireFrappliance/MetaDataImporting/SapphireMovieImporter.m

    r732 r739  
    184184        NSError *error = nil ; 
    185185        NSURL * url=[NSURL URLWithString:[NSString stringWithFormat:@"http://www.imdb.com%@/posters",candidateIMDBLink]] ; 
    186         NSXMLDocument *document = [[NSXMLDocument alloc] initWithContentsOfURL:url options:NSXMLDocumentTidyHTML error:&error]; 
     186        NSXMLDocument *document = [[[NSXMLDocument alloc] initWithContentsOfURL:url options:NSXMLDocumentTidyHTML error:&error] autorelease]; 
    187187        NSXMLElement *root = [document rootElement]; 
    188188 
    … …  
    220220        NSError *error = nil ; 
    221221        NSURL * url=[NSURL URLWithString:[NSString stringWithFormat:@"http://www.IMPAwards.com%@",posterPageLink]] ; 
    222         NSXMLDocument *document = [[NSXMLDocument alloc] initWithContentsOfURL:url options:NSXMLDocumentTidyHTML error:&error]; 
     222        NSXMLDocument *document = [[[NSXMLDocument alloc] initWithContentsOfURL:url options:NSXMLDocumentTidyHTML error:&error] autorelease]; 
    223223        NSXMLElement *root = [document rootElement]; 
    224224        NSMutableArray * candidatePosterLinks=[NSMutableArray arrayWithObjects:nil] ; 
    … …  
    239239                        yearPathComponent=[newPosterPageLink stringByDeletingLastPathComponent]; 
    240240                        url=[NSURL URLWithString:[NSString stringWithFormat:@"http://www.IMPAwards.com%@",newPosterPageLink]] ; 
    241                         document = [[NSXMLDocument alloc] initWithContentsOfURL:url options:NSXMLDocumentTidyHTML error:&error]; 
     241                        document = [[[NSXMLDocument alloc] initWithContentsOfURL:url options:NSXMLDocumentTidyHTML error:&error] autorelease]; 
    242242                        root = [document rootElement]; 
    243243                        results = [root objectsForXQuery:IMP_POSTER_CANDIDATES_XPATH error:&error];                      
    … …  
    318318        /*Get the movie html*/ 
    319319        NSURL *url = [NSURL URLWithString:[NSString stringWithFormat:@"http://www.IMDB.com%@",movieTitleLink]]; 
    320         NSXMLDocument *document = [[NSXMLDocument alloc] initWithContentsOfURL:url options:NSXMLDocumentTidyHTML error:&error]; 
     320        NSXMLDocument *document = [[[NSXMLDocument alloc] initWithContentsOfURL:url options:NSXMLDocumentTidyHTML error:&error] autorelease]; 
    321321         
    322322        /* Get the movie title */ 
    … …  
    354354        NSNumber * oscarsWon=nil ; 
    355355        NSArray * directors=nil; 
    356         NSArray * writers=nil; 
     356//      NSArray * writers=nil; 
    357357        NSArray * genres=nil; 
    358358        if([rawData count]) 
    … …  
    375375                                         
    376376                                } 
    377                                 else if([dataType hasPrefix:@"Writer"]) 
    378                                 { 
    379                                         NSString *writersStr = [[trimmer string] substringFromIndex:[trimmer scanLocation]+1]; 
    380                                         NSMutableArray *mutDirs = [[writersStr componentsSeparatedByString:@"\n"] mutableCopy]; 
    381                                         [mutDirs removeObject:@""]; 
    382                                         int i, count = [mutDirs count]; 
    383                                         for(i=0; i<count; i++) 
    384                                         { 
    385                                                 NSString *tdirector; 
    386                                                 NSScanner *typeTrimmer = [[NSScanner alloc] initWithString:[mutDirs objectAtIndex:i]]; 
    387                                                 [typeTrimmer scanUpToString:@" (" intoString:&tdirector]; 
    388                                                 [mutDirs replaceObjectAtIndex:i withObject:tdirector]; 
    389                                                 [typeTrimmer release]; 
    390                                         } 
    391                                         writers = [[mutDirs copy] autorelease]; 
    392                                         [mutDirs release]; 
    393                                 } 
     377//                              else if([dataType hasPrefix:@"Writer"]) 
     378//                              { 
     379//                                      NSString *writersStr = [[trimmer string] substringFromIndex:[trimmer scanLocation]+1]; 
     380//                                      NSMutableArray *mutDirs = [[writersStr componentsSeparatedByString:@"\n"] mutableCopy]; 
     381//                                      [mutDirs removeObject:@""]; 
     382//                                      int i, count = [mutDirs count]; 
     383//                                      for(i=0; i<count; i++) 
     384//                                      { 
     385//                                              NSString *tdirector; 
     386//                                              NSScanner *typeTrimmer = [[NSScanner alloc] initWithString:[mutDirs objectAtIndex:i]]; 
     387//                                              [typeTrimmer scanUpToString:@" (" intoString:&tdirector]; 
     388//                                              [mutDirs replaceObjectAtIndex:i withObject:tdirector]; 
     389//                                              [typeTrimmer release]; 
     390//                                      } 
     391//                                      writers = [[mutDirs copy] autorelease]; 
     392//                                      [mutDirs release]; 
     393//                              } 
    394394                                else if([dataType hasPrefix:@"Director"]) 
    395395                                { 
    … …  
    420420                                        } 
    421421                                        else if([awardsStr hasPrefix:@"Won Oscar"]) 
    422                                                 oscarsWon=[[NSNumber alloc] initWithInt:1]; 
     422                                                oscarsWon=[NSNumber numberWithInt:1]; 
    423423                                         
    424424                                } 
    … …  
    564564        NSArray * results = nil; 
    565565        NSMutableArray *ret=nil; 
    566         NSXMLDocument *document = [[NSXMLDocument alloc] initWithContentsOfURL:url options:NSXMLDocumentTidyHTML error:&error]; 
     566        NSXMLDocument *document = [[[NSXMLDocument alloc] initWithContentsOfURL:url options:NSXMLDocumentTidyHTML error:&error] autorelease]; 
    567567        NSXMLElement *root = [document rootElement];     
    568568        NSString *resultTitle=[[[root objectsForXQuery:@"//title" error:&error]objectAtIndex:0] stringValue]; 

  • branches/CoreData/SapphireFrappliance/MetaDataImporting/SapphirePosterChooser.m

    r707 r739  
    451451        [asset release]; 
    452452         
    453         return preview; 
     453        return [preview autorelease]; 
    454454} 
    455455 

  • branches/CoreData/SapphireFrappliance/MetaDataImporting/SapphireTVShowImporter.m

    r732 r739  
    199199        NSURL *url = [NSURL URLWithString:[NSString stringWithFormat:@"http://www.tvrage.com%@/episode_guide/%d", seriesName, season]]; 
    200200        NSError *error = nil; 
    201         NSXMLDocument *document = [[NSXMLDocument alloc] initWithContentsOfURL:url options:NSXMLDocumentTidyHTML error:&error]; 
     201        NSXMLDocument *document = [[[NSXMLDocument alloc] initWithContentsOfURL:url options:NSXMLDocumentTidyHTML error:&error] autorelease]; 
    202202        /* Dump XML document to disk (Dev Only) */ 
    203203/*      NSString *documentPath =[NSHomeDirectory() stringByAppendingPathComponent:@"Library/Application Support/Sapphire/XML"]; 
    … …  
    343343        NSURL *url = [NSURL URLWithString:[NSString stringWithFormat:@"http://www.tvrage.com/search.php?search=%@&sonly=1", [searchStr URLEncode]]]; 
    344344        NSError *error = nil; 
    345         NSXMLDocument *document = [[NSXMLDocument alloc] initWithContentsOfURL:url options:NSXMLDocumentTidyHTML error:&error]; 
     345        NSXMLDocument *document = [[[NSXMLDocument alloc] initWithContentsOfURL:url options:NSXMLDocumentTidyHTML error:&error] autorelease]; 
    346346        SapphireLog(SAPPHIRE_LOG_IMPORT, SAPPHIRE_LOG_LEVEL_DEBUG, @"Document is %@", document); 
    347347        if(error != nil && ![[error domain] isEqualToString:@"NSXMLParserErrorDomain"]) 
    … …  
    641641                NSURLRequest *request = [NSURLRequest requestWithURL:imageURL]; 
    642642                SapphireTVShowDataMenuDownloadDelegate *myDelegate = [[SapphireTVShowDataMenuDownloadDelegate alloc] initWithDest:imageDestination]; 
    643                 [[NSURLDownload alloc] initWithRequest:request delegate:myDelegate]; 
     643                [[[NSURLDownload alloc] initWithRequest:request delegate:myDelegate] autorelease]; 
    644644                [myDelegate release]; 
    645645        } 

  • branches/CoreData/SapphireFrappliance/MetaDataImporting/SapphireXMLFileDataImporter.m

    r737 r739  
    8686} 
    8787 
     88- (void) dealloc 
     89{ 
     90        [super dealloc]; 
     91} 
     92 
    8893- (void)setImporterDataMenu:(SapphireImporterDataMenu *)theDataMenu 
    8994{ 
    … …  
    114119        NSError *error = nil; 
    115120        NSMutableDictionary *newMetaData=[NSMutableDictionary dictionary]; 
    116         NSXMLDocument *document = [[NSXMLDocument alloc] initWithContentsOfURL:url options:NSXMLDocumentTidyXML error:&error]; 
     121        NSXMLDocument *document = [[[NSXMLDocument alloc] initWithContentsOfURL:url options:NSXMLDocumentTidyXML error:&error] autorelease]; 
    117122        NSXMLElement *root = [document rootElement]; 
    118123        if(!root) 
    … …  
    131136                if(fclass != FILE_CLASS_UNKNOWN) 
    132137                        [newMetaData setObject:[NSNumber numberWithInt:fclass] forKey:FILE_CLASS_KEY]; 
    133                 else 
    134                         fclass = [metaData fileClassValue]; 
    135138        } 
    136139                 

  • branches/CoreData/SapphireFrappliance/Players/SapphireAudioMedia.m

    r541 r739  
    120120} 
    121121 
    122 - (int)duration 
     122- (long)duration 
    123123{ 
    124124        QTTime duration = [movie duration]; 

  • branches/CoreData/SapphireFrappliance/Players/SapphireAudioPlayer.m

    r541 r739  
    6363{ 
    6464        [super setMedia:media inTracklist:tracklist error:error]; 
    65         if(*error == nil) 
     65        if(error != NULL && *error == nil) 
    6666        { 
    6767                movie = [[QTMovie alloc] initWithURL:[NSURL URLWithString:[media mediaURL]] error:error]; 

  • branches/CoreData/SapphireFrappliance/Players/SapphireVideoPlayer.m

    r696 r739  
    334334} 
    335335 
    336 - (double)elapsedPlaybackTime 
     336- (float)elapsedPlaybackTime 
    337337{ 
    338338        if([[SapphireVideoPlayer superclass] instancesRespondToSelector:@selector(elapsedPlaybackTime)]) 

  • branches/CoreData/SapphireFrappliance/Settings/SapphireCollectionSettings.m

    r696 r739  
    188188        SapphireMediaPreview *preview = [[SapphireMediaPreview alloc] initWithScene:[self scene]]; 
    189189        [preview setUtilityData:settingMeta]; 
     190        [settingMeta release]; 
    190191        [preview setShowsMetadataImmediately:YES]; 
    191192        /*And go*/ 

  • branches/CoreData/SapphireFrappliance/Settings/SapphireRadioSetting.m

    r541 r739  
    146146                SapphireMediaPreview *preview = [[SapphireMediaPreview alloc] initWithScene:[self scene]]; 
    147147                [preview setUtilityData:settingMeta]; 
     148                [settingMeta release]; 
    148149                [preview setShowsMetadataImmediately:YES]; 
    149150                /*And go*/ 

  • branches/CoreData/SapphireFrappliance/Settings/SapphireSettings.m

    r708 r739  
    520520                SapphireMediaPreview *preview = [[SapphireMediaPreview alloc] initWithScene:[self scene]]; 
    521521                [preview setUtilityData:settingMeta]; 
     522                [settingMeta release]; 
    522523                [preview setShowsMetadataImmediately:YES]; 
    523524                /*And go*/ 

  • branches/CoreData/SapphireFrappliance/Settings/SapphireTheme.m

    r592 r739  
    110110 * @return A CGImageRef (retained) from the path 
    111111 */ 
    112 - (CGImageRef)loadImage:(NSString *)path 
     112- (CGImageRef)newImage:(NSString *)path 
    113113{ 
    114114        NSString *bundlePath = [[NSBundle bundleForClass:[self class]] bundlePath]; 
    … …  
    141141        } 
    142142        /*Load it*/ 
    143         CGImageRef image = [self loadImage:[gemFiles objectForKey:type]]; 
     143        CGImageRef image = [self newImage:[gemFiles objectForKey:type]]; 
    144144        if(image != NULL) 
    145145        {