Changeset 424

Timestamp:

Jun 21, 2006, 11:46:41 PM (16 years ago)

Author:

livings124

Message:

Make method names less generic to avoid conflicts.
Use methods in the Torrent class whenever possible.

Location:

trunk/macosx

Files:

• Badger.m (modified) (5 diffs)
• Controller.m (modified) (7 diffs)
• StringAdditions.m (modified) (1 diff)
• Torrent.h (modified) (2 diffs)
• Torrent.m (modified) (8 diffs)
• TorrentTableView.m (modified) (2 diffs)

trunk/macosx/Badger.m

@@ -45 +45 @@
         
         NSShadow * stringShadow = [[NSShadow alloc] init];
-        [stringShadow setShadowOffset: NSMakeSize(2, -2)];
-        [stringShadow setShadowBlurRadius: 4];
+        [stringShadow setShadowOffset: NSMakeSize(2.0, -2.0)];
+        [stringShadow setShadowBlurRadius: 4.0];
         
         fAttributes = [[NSDictionary alloc] initWithObjectsAndKeys:
             [NSColor whiteColor], NSForegroundColorAttributeName,
-            [NSFont fontWithName: @"Helvetica-Bold" size: 28], NSFontAttributeName,
+            [NSFont fontWithName: @"Helvetica-Bold" size: 28.0], NSFontAttributeName,
             stringShadow, NSShadowAttributeName, nil];
         
@@ -66 +66 @@
     [fDockIcon release];
     [fBadgedDockIcon release];
-
     [fAttributes release];
 
@@ -83 +82 @@
     {
         fCompleted = completed;
-        
         dockIcon = [fDockIcon copy];
         
@@ -126 +124 @@
     if (speedShown)
     {
-        NSRect badgeRect, stringRect;
+        NSRect badgeRect;
         badgeRect.size = [fUploadBadge size];
         badgeRect.origin = NSZeroPoint;
         
         //ignore shadow of badge when placing string
+        NSRect stringRect = badgeRect;
         float badgeBottomExtra = 2.0;
-        stringRect = badgeRect;
         stringRect.size.height -= badgeBottomExtra;
         stringRect.origin.y += badgeBottomExtra;
@@ -178 +176 @@
             
         [NSApp setApplicationIconImage: dockIcon];
-        
         [dockIcon release];
     }

trunk/macosx/Controller.m

@@ -320 +320 @@
         [torrent setDownloadFolder: [[openPanel filenames] objectAtIndex: 0]];
         if ([fDefaults boolForKey: @"AutoStartDownload"])
-            [torrent start];
+            [torrent startTransfer];
         [fTorrents addObject: torrent];
         
@@ -372 +372 @@
             [torrent setDownloadFolder: folder];
             if (autoStart)
-                [torrent start];
+                [torrent startTransfer];
             [fTorrents addObject: torrent];
         }
@@ -462 +462 @@
 - (void) resumeTorrentWithIndex: (NSIndexSet *) indexSet
 {
-    Torrent * torrent;
     unsigned int i;
     for (i = [indexSet firstIndex]; i != NSNotFound; i = [indexSet indexGreaterThanIndex: i])
-        [[fTorrents objectAtIndex: i] start];
+        [[fTorrents objectAtIndex: i] startTransfer];
     
     [self updateUI: nil];
@@ -485 +484 @@
 {
     unsigned int i;
-    Torrent * torrent;
     for (i = [indexSet firstIndex]; i != NSNotFound; i = [indexSet indexGreaterThanIndex: i])
-    {
-        torrent = [fTorrents objectAtIndex: i];
-        [torrent stop];
-    }
+        [[fTorrents objectAtIndex: i] stopTransfer];
     
     [self updateUI: nil];
@@ -540 +535 @@
         }
 
-        NSBeginAlertSheet(title,
-            @"Remove", @"Cancel", nil, fWindow, self,
+        NSBeginAlertSheet(title, @"Remove", @"Cancel", nil, fWindow, self,
             @selector(removeSheetDidEnd:returnCode:contextInfo:), nil, dict, message);
     }
@@ -571 +565 @@
     while ((torrent = [enumerator nextObject]))
     {
-        [torrent stop];
+        [torrent stopTransfer];
 
         if (deleteData)
@@ -670 +664 @@
 - (void) revealFile: (id) sender
 {
-    Torrent * torrent;
     NSIndexSet * indexSet = [fTableView selectedRowIndexes];
     unsigned int i;
     
     for (i = [indexSet firstIndex]; i != NSNotFound; i = [indexSet indexGreaterThanIndex: i])
-    {
-        torrent = [fTorrents objectAtIndex: i];
-        [torrent reveal];
-    }
+        [[fTorrents objectAtIndex: i] revealData];
 }
 

trunk/macosx/StringAdditions.m

@@ -65 +65 @@
     }
 
-    NSString * value;
+    NSString * sizeString;
     //attempt to have values with 3 digits
     if (convertedSize < 10.0)
-        value = [NSString stringWithFormat: @"%.2f", convertedSize];
+        sizeString = [NSString stringWithFormat: @"%.2f", convertedSize];
     else if (convertedSize < 100.0)
-        value = [NSString stringWithFormat: @"%.1f", convertedSize];
+        sizeString = [NSString stringWithFormat: @"%.1f", convertedSize];
     else
-        value = [NSString stringWithFormat: @"%.0f", convertedSize];
+        sizeString = [NSString stringWithFormat: @"%.0f", convertedSize];
 
-    return [value stringByAppendingString: unit];
+    return [sizeString stringByAppendingString: unit];
 }
 

trunk/macosx/Torrent.h

@@ -63 +63 @@
 
 - (void)        update;
-- (void)        start;
-- (void)        stop;
+- (void)        startTransfer;
+- (void)        stopTransfer;
 - (void)        removeForever;
 - (void)        sleep;
@@ -75 +75 @@
 - (void)        setRatioLimit: (float) limit;
 
-- (void)    reveal;
+- (void)    revealData;
 - (void)    trashData;
 - (void)    trashTorrent;

trunk/macosx/Torrent.m

@@ -72 +72 @@
         NSString * paused;
         if (!(paused = [history objectForKey: @"Paused"]) || [paused isEqualToString: @"NO"])
-            [self start];
+            [self startTransfer];
     }
     return self;
@@ -138 +138 @@
             && [self ratio] >= [fDefaults floatForKey: @"RatioLimit"])))
     {
-        [self stop];
+        [self stopTransfer];
         [self setStopRatioSetting: RATIO_NO_CHECK];
         fFinishedSeeding = YES;
@@ -217 +217 @@
 }
 
-- (void) start
+- (void) startTransfer
 {
     if (![self isActive])
@@ -226 +226 @@
 }
 
-- (void) stop
+- (void) stopTransfer
 {
     if ([self isActive])
@@ -234 +234 @@
 - (void) removeForever
 {
-    if (fInfo->flags & TR_FSAVEPRIVATE)
+    if (fPrivateTorrent)
         tr_torrentRemoveSaved(fHandle);
 }
@@ -241 +241 @@
 {
     if ((fResumeOnWake = [self isActive]))
-        [self stop];
+        [self stopTransfer];
 }
 
@@ -247 +247 @@
 {
     if (fResumeOnWake)
-        [self start];
+        [self startTransfer];
 }
 
@@ -277 +277 @@
 }
 
-- (void) reveal
+- (void) revealData
 {
     [[NSWorkspace sharedWorkspace] selectFile: [self dataLocation] inFileViewerRootedAtPath: nil];

trunk/macosx/TorrentTableView.m

@@ -55 +55 @@
         fRevealOnIcon = [NSImage imageNamed: @"RevealOn.png"];
         fRevealOffIcon = [NSImage imageNamed: @"RevealOff.png"];
+        
+        fClickPoint = NSZeroPoint;
     }
     
@@ -104 +106 @@
     }
     else if (sameRow && [self pointInRevealRect: point] && [self pointInRevealRect: fClickPoint])
-        [[fTorrents objectAtIndex: row] reveal];
+        [[fTorrents objectAtIndex: row] revealData];
         else if ([event clickCount] == 2)
     {
         if ([self pointInIconRect: point])
-            [[fTorrents objectAtIndex: row] reveal];
+            [[fTorrents objectAtIndex: row] revealData];
         else
             [fController showInfo: nil];