Changeset 5236

Timestamp:

Mar 11, 2008, 3:46:38 AM (14 years ago)

Author:

livings124

Message:

make the file menu code a bit more straightforward

File:

• trunk/macosx/TorrentTableView.m (modified) (4 diffs)

trunk/macosx/TorrentTableView.m

@@ -530 +530 @@
         return;
     
-    //get and update file menu
+    NSInteger numberOfNonFileItems = [fActionMenu numberOfItems];
+    
+    //update file action menu
     fMenuTorrent = [[self itemAtRow: row] retain];
-    
-    NSMenu * fileMenu = [[NSMenu alloc] initWithTitle: @"TorrentMenu"];
-    [fileMenu setAutoenablesItems: NO];
-    [self createFileMenu: fileMenu forFiles: [fMenuTorrent fileList]];
-    
-    //add file menu items to action menu
-    NSInteger numberOfItems = [fileMenu numberOfItems];
-    [fActionMenu appendItemsFromMenu: fileMenu atIndexes: [NSIndexSet indexSetWithIndexesInRange: NSMakeRange(0, numberOfItems)]
-        atBottom: YES];
-    [fileMenu release];
+    [self createFileMenu: fActionMenu forFiles: [fMenuTorrent fileList]];
     
     //place menu below button
@@ -556 +549 @@
     
     NSInteger i;
-    for (i = 0; i < numberOfItems; i++)
-        [fActionMenu removeItemAtIndex: [fActionMenu numberOfItems]-1];
+    for (i = [fActionMenu numberOfItems]-1; i >= numberOfNonFileItems; i--)
+        [fActionMenu removeItemAtIndex: i];
     
     [fMenuTorrent release];
@@ -637 +630 @@
     else  //assume the menu is part of the file list
     {
+        if ([menu numberOfItems] > 0)
+            return;
+        
         NSMenu * supermenu = [menu supermenu];
         [self createFileMenu: menu forFiles: [[[supermenu itemAtIndex: [supermenu indexOfItemWithSubmenu: menu]]
@@ -827 +823 @@
 - (void) createFileMenu: (NSMenu *) menu forFiles: (NSArray *) files
 {
-    if ([menu numberOfItems] > 0)
-        return;
-    
     NSEnumerator * enumerator = [files objectEnumerator];
     NSDictionary * dict;