OSDN Git Service

MacGui: - Implement automatic extension switching for mp4/m4v.
[handbrake-jp/handbrake-jp-git.git] / macosx / Controller.mm
index 11cb254..c1a589b 100644 (file)
@@ -16,7 +16,6 @@
 #include "dvdread/dvd_reader.h"
 #include "HBPresets.h"
 
-#define _(a) NSLocalizedString(a,NULL)
 #define DragDropSimplePboardType       @"MyCustomOutlineViewPboardType"
 
 /* We setup the toolbar values here */
@@ -34,26 +33,31 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
  *******************************/
 @implementation HBController
 
-- init
+- (id)init
 {
     self = [super init];
+    if( !self )
+    {
+        return nil;
+    }
+
     [HBPreferencesController registerUserDefaults];
     fHandle = NULL;
+
     /* Check for check for the app support directory here as
-        * outputPanel needs it right away, as may other future methods
-        */
-    /* We declare the default NSFileManager into fileManager */
-       NSFileManager * fileManager = [NSFileManager defaultManager];
-       /* we set the files and support paths here */
-       AppSupportDirectory = @"~/Library/Application Support/HandBrake";
-    AppSupportDirectory = [AppSupportDirectory stringByExpandingTildeInPath];
-    /* We check for the app support directory for handbrake */
-       if ([fileManager fileExistsAtPath:AppSupportDirectory] == 0) 
-       {
-               // If it doesnt exist yet, we create it here 
-               [fileManager createDirectoryAtPath:AppSupportDirectory attributes:nil];
-       }
-    
+     * outputPanel needs it right away, as may other future methods
+     */
+    NSString *libraryDir = [NSSearchPathForDirectoriesInDomains( NSLibraryDirectory,
+                                                                 NSUserDomainMask,
+                                                                 YES ) objectAtIndex:0];
+    AppSupportDirectory = [[libraryDir stringByAppendingPathComponent:@"Application Support"]
+                                       stringByAppendingPathComponent:@"HandBrake"];
+    if( ![[NSFileManager defaultManager] fileExistsAtPath:AppSupportDirectory] )
+    {
+        [[NSFileManager defaultManager] createDirectoryAtPath:AppSupportDirectory
+                                                   attributes:nil];
+    }
+
     outputPanel = [[HBOutputPanelController alloc] init];
     fPictureController = [[PictureController alloc] initWithDelegate:self];
     fQueueController = [[HBQueueController alloc] init];
@@ -76,36 +80,37 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
 {
     /* Init libhb with check for updates libhb style set to "0" so its ignored and lets sparkle take care of it */
     fHandle = hb_init(HB_DEBUG_ALL, 0);
-    
+
        // Set the Growl Delegate
-    [GrowlApplicationBridge setGrowlDelegate: self];    
+    [GrowlApplicationBridge setGrowlDelegate: self];
     /* Init others controllers */
     [fPictureController SetHandle: fHandle];
     [fQueueController   setHandle: fHandle];
     [fQueueController   setHBController: self];
-       
+
     fChapterTitlesDelegate = [[ChapterTitles alloc] init];
     [fChapterTable setDataSource:fChapterTitlesDelegate];
-    
+    [fChapterTable setDelegate:fChapterTitlesDelegate];
+
     /* Call UpdateUI every 1/2 sec */
-    [[NSRunLoop currentRunLoop] addTimer: [NSTimer
-                                           scheduledTimerWithTimeInterval: 0.5 target: self
-                                           selector: @selector( updateUI: ) userInfo: NULL repeats: YES]
-                                 forMode: NSEventTrackingRunLoopMode];
-    
+    [[NSRunLoop currentRunLoop] addTimer:[NSTimer
+                                          scheduledTimerWithTimeInterval:0.5 target:self
+                                          selector:@selector(updateUI:) userInfo:nil repeats:YES]
+                                 forMode:NSEventTrackingRunLoopMode];
+
     // Open debug output window now if it was visible when HB was closed
     if ([[NSUserDefaults standardUserDefaults] boolForKey:@"OutputPanelIsOpen"])
         [self showDebugOutputPanel:nil];
-    
+
     // Open queue window now if it was visible when HB was closed
     if ([[NSUserDefaults standardUserDefaults] boolForKey:@"QueueWindowIsOpen"])
         [self showQueueWindow:nil];
-    
+
        [self openMainWindow:nil];
-    
+
     /* Show Browse Sources Window ASAP */
-    [self performSelectorOnMainThread: @selector(browseSources:)
-                           withObject: NULL waitUntilDone: NO];
+    [self performSelectorOnMainThread:@selector(browseSources:)
+                           withObject:nil waitUntilDone:NO];
 }
 
 - (NSApplicationTerminateReply) applicationShouldTerminate: (NSApplication *) app
@@ -171,12 +176,12 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
     /* Initialize currentScanCount so HB can use it to
                evaluate successive scans */
        currentScanCount = 0;
-       
+
     /* Init UserPresets .plist */
        [self loadPresets];
-               
+
        fRipIndicatorShown = NO;  // initially out of view in the nib
-       
+
        /* Show/Dont Show Presets drawer upon launch based
                on user preference DefaultPresetsDrawerShow*/
        if ([[NSUserDefaults standardUserDefaults] boolForKey:@"DefaultPresetsDrawerShow"] > 0)
@@ -202,39 +207,39 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
     menuItem = [[fDstFormatPopUp menu] addItemWithTitle:@"OGM file" action: NULL keyEquivalent: @""];
     [menuItem setTag: HB_MUX_OGM];
     [fDstFormatPopUp selectItemAtIndex: 0];
-       
-    [self formatPopUpChanged: NULL];
-    
-       /* We enable the create chapters checkbox here since we are .mp4 */     
+
+    [self formatPopUpChanged:nil];
+
+       /* We enable the create chapters checkbox here since we are .mp4 */
        [fCreateChapterMarkers setEnabled: YES];
        if ([fDstFormatPopUp indexOfSelectedItem] == 0 && [[NSUserDefaults standardUserDefaults] boolForKey:@"DefaultChapterMarkers"] > 0)
        {
                [fCreateChapterMarkers setState: NSOnState];
        }
-       
-       
-       
-       
+
+
+
+
     [fDstFile2Field setStringValue: [NSString stringWithFormat:
         @"%@/Desktop/Movie.mp4", NSHomeDirectory()]];
-       
+
     /* Video encoder */
     [fVidEncoderPopUp removeAllItems];
     [fVidEncoderPopUp addItemWithTitle: @"FFmpeg"];
     [fVidEncoderPopUp addItemWithTitle: @"XviD"];
-       
-    
-       
+
+
+
     /* Video quality */
     [fVidTargetSizeField setIntValue: 700];
        [fVidBitrateField    setIntValue: 1000];
-       
+
     [fVidQualityMatrix   selectCell: fVidBitrateCell];
-    [self videoMatrixChanged: NULL];
-       
+    [self videoMatrixChanged:nil];
+
     /* Video framerate */
     [fVidRatePopUp removeAllItems];
-       [fVidRatePopUp addItemWithTitle: _( @"Same as source" )];
+       [fVidRatePopUp addItemWithTitle: NSLocalizedString( @"Same as source", @"" )];
     for( int i = 0; i < hb_video_rates_count; i++ )
     {
         if ([[NSString stringWithCString: hb_video_rates[i].string] isEqualToString: [NSString stringWithFormat: @"%.3f",23.976]])
@@ -284,41 +289,21 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
        
     /* Bottom */
     [fStatusField setStringValue: @""];
-       
+
     [self enableUI: NO];
        [self setupToolbar];
-       
-       [fPresetsActionButton setMenu:fPresetsActionMenu];
-       
+
        /* We disable the Turbo 1st pass checkbox since we are not x264 */
        [fVidTurboPassCheck setEnabled: NO];
        [fVidTurboPassCheck setState: NSOffState];
-       
-       
+
+
        /* lets get our default prefs here */
-       [self getDefaultPresets: NULL];
+       [self getDefaultPresets:nil];
        /* lets initialize the current successful scancount here to 0 */
        currentSuccessfulScanCount = 0;
-    
-}
-
-- (void) TranslateStrings
-{
-    [fSrcTitleField     setStringValue: _( @"Title:" )];
-    [fSrcChapterField   setStringValue: _( @"Chapters:" )];
-    [fSrcChapterToField setStringValue: _( @"to" )];
-    [fSrcDuration1Field setStringValue: _( @"Duration:" )];
-
-    [fDstFormatField    setStringValue: _( @"Format:" )];
-    [fDstFile1Field     setStringValue: _( @"File:" )];
-    [fDstBrowseButton   setTitle:       _( @"Browse" )];
-
-    [fVidRateField      setStringValue: _( @"Framerate (fps):" )];
-    [fVidEncoderField   setStringValue: _( @"Encoder:" )];
-    [fVidQualityField   setStringValue: _( @"Quality:" )];
 }
 
-
 - (void) enableUI: (bool) b
 {
     NSControl * controls[] =
@@ -343,7 +328,7 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
                fPicLabelAr,fPicLabelDeinterlace,fPicSettingPAR,fPicLabelAnamorphic,fPresetsAdd,fPresetsDelete,
                fCreateChapterMarkers,fVidTurboPassCheck,fDstMp4LargeFileCheck,fPicLabelAutoCrop,
                fPicSettingAutoCrop,fPicSettingDetelecine,fPicLabelDetelecine,fPicLabelDenoise,fPicSettingDenoise,
-        fSubForcedCheck,fPicSettingDeblock,fPicLabelDeblock,fPresetsOutlineView,
+        fSubForcedCheck,fPicSettingDeblock,fPicLabelDeblock,fPicLabelDecomb,fPicSettingDecomb,fPresetsOutlineView,
         fAudDrcLabel,fDstMp4HttpOptFileCheck,fDstMp4iPodFileCheck};
 
     for( unsigned i = 0;
@@ -362,22 +347,23 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
         [controls[i] setEnabled: b];
 
     }
-       
+
        if (b) {
 
         /* if we're enabling the interface, check if the audio mixdown controls need to be enabled or not */
         /* these will have been enabled by the mass control enablement above anyway, so we're sense-checking it here */
-        [self setEnabledStateOfAudioMixdownControls: NULL];
+        [self setEnabledStateOfAudioMixdownControls:nil];
         /* we also call calculatePictureSizing here to sense check if we already have vfr selected */
-        [self calculatePictureSizing: NULL];
-       
+        [self calculatePictureSizing:nil];
+        [self shouldEnableHttpMp4CheckBox: nil];
+
        } else {
 
                [fPresetsOutlineView setEnabled: NO];
-       
+
        }
 
-    [self videoMatrixChanged: NULL];
+    [self videoMatrixChanged:nil];
     [fAdvancedOptions enableUI:b];
 }
 
@@ -477,46 +463,45 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
        int checkScanCount = hb_get_scancount( fHandle );
        if (checkScanCount > currentScanCount)
        {
-               
                currentScanCount = checkScanCount;
         [fScanIndicator setIndeterminate: NO];
         [fScanIndicator setDoubleValue: 0.0];
         [fScanIndicator setHidden: YES];
-               [self showNewScan: NULL];
+               [self showNewScan:nil];
        }
-       
+
     hb_state_t s;
     hb_get_state( fHandle, &s );
-       
-       
+
+
     switch( s.state )
     {
         case HB_STATE_IDLE:
                break;
-#define p s.param.scanning                     
+#define p s.param.scanning
         case HB_STATE_SCANNING:
                {
             [fSrcDVD2Field setStringValue: [NSString stringWithFormat:
-                                            _( @"Scanning title %d of %d..." ),
+                                            NSLocalizedString( @"Scanning title %d of %d...", @"" ),
                                             p.title_cur, p.title_count]];
             [fScanIndicator setHidden: NO];
             [fScanIndicator setDoubleValue: 100.0 * ( p.title_cur - 1 ) / p.title_count];
             break;
                }
 #undef p
-       
+
 #define p s.param.scandone
         case HB_STATE_SCANDONE:
         {
             [fScanIndicator setIndeterminate: NO];
             [fScanIndicator setDoubleValue: 0.0];
             [fScanIndicator setHidden: YES];
-                       [self showNewScan: NULL];
-            [toolbar validateVisibleItems];
+                       [self showNewScan:nil];
+            [[fWindow toolbar] validateVisibleItems];
                        break;
         }
 #undef p
-                       
+
 #define p s.param.working
         case HB_STATE_WORKING:
         {
@@ -526,12 +511,12 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
                                jobs during encoding, if they cannot be fixed in libhb, will implement a
                                nasty but working cocoa solution */
                        /* Update text field */
-                       string = [NSMutableString stringWithFormat: _( @"Encoding: task %d of %d, %.2f %%" ), p.job_cur, p.job_count, 100.0 * p.progress];
+                       string = [NSMutableString stringWithFormat: NSLocalizedString( @"Encoding: task %d of %d, %.2f %%", @"" ), p.job_cur, p.job_count, 100.0 * p.progress];
             
                        if( p.seconds > -1 )
             {
                 [string appendFormat:
-                    _( @" (%.2f fps, avg %.2f fps, ETA %02dh%02dm%02ds)" ),
+                    NSLocalizedString( @" (%.2f fps, avg %.2f fps, ETA %02dh%02dm%02ds)", @"" ),
                     p.rate_cur, p.rate_avg, p.hours, p.minutes, p.seconds];
             }
             [fStatusField setStringValue: string];
@@ -577,7 +562,7 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
                        
             /* Update text field */
             string = [NSMutableString stringWithFormat:
-                _( @"Muxing..." )];
+                NSLocalizedString( @"Muxing...", @"" )];
             [fStatusField setStringValue: string];
                        
             /* Update slider */
@@ -595,7 +580,7 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
 #undef p
                        
         case HB_STATE_PAUSED:
-                   [fStatusField setStringValue: _( @"Paused" )];
+                   [fStatusField setStringValue: NSLocalizedString( @"Paused", @"" )];
             
                        // Pass along the info to HBQueueController
             [fQueueController libhbStateChanged: s];
@@ -607,16 +592,16 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
             // HB_STATE_WORKDONE happpens as a result of libhb finishing all its jobs
             // or someone calling hb_stop. In the latter case, hb_stop does not clear
             // out the remaining passes/jobs in the queue. We'll do that here.
-                        
+
             // Delete all remaining jobs of this encode.
             hb_job_t * job;
             while( ( job = hb_job( fHandle, 0 ) ) && ( !IsFirstPass(job->sequence_id) ) )
                 hb_rem( fHandle, job );
 
-            [fStatusField setStringValue: _( @"Done." )];
+            [fStatusField setStringValue: NSLocalizedString( @"Done.", @"" )];
             [fRipIndicator setIndeterminate: NO];
             [fRipIndicator setDoubleValue: 0.0];
-            [toolbar validateVisibleItems];
+            [[fWindow toolbar] validateVisibleItems];
 
             /* Restore dock icon */
             [self UpdateDockIcon: -1.0];
@@ -638,7 +623,7 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
             /* Check to see if the encode state has not been cancelled
                                to determine if we should check for encode done notifications */
                        if (fEncodeState != 2)                  {
-                               /* If Alert Window or Window and Growl has been selected */
+                /* If Alert Window or Window and Growl has been selected */
                                if ([[[NSUserDefaults standardUserDefaults] stringForKey:@"AlertWhenDone"] isEqualToString: @"Alert Window"] || 
                                        [[[NSUserDefaults standardUserDefaults] stringForKey:@"AlertWhenDone"] isEqualToString: @"Alert Window And Growl"])
                 {
@@ -647,45 +632,32 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
                                        NSBeep();
                                        status = NSRunAlertPanel(@"Put down that cocktail...",@"Your HandBrake encode is done!", @"OK", nil, nil);
                                        [NSApp requestUserAttention:NSCriticalRequest];
-                                       if ( status == NSAlertDefaultReturn ) 
-                                       {
-                                               [self enableUI: YES];
-                                       }
+
                 }
-                               else
-                               {
-                                       [self enableUI: YES];
-                               }
-                                  /* If sleep has been selected */ 
-            if ([[[NSUserDefaults standardUserDefaults] stringForKey:@"AlertWhenDone"] isEqualToString: @"Put Computer To Sleep"]) 
-                { 
-               /* Sleep */ 
-               NSDictionary* errorDict; 
-               NSAppleEventDescriptor* returnDescriptor = NULL; 
-               NSAppleScript* scriptObject = [[NSAppleScript alloc] initWithSource: 
-                        @"tell application \"Finder\" to sleep"]; 
-               returnDescriptor = [scriptObject executeAndReturnError: &errorDict]; 
-               [scriptObject release]; 
-               [self enableUI: YES]; 
-                } 
-            /* If Shutdown has been selected */ 
-            if ([[[NSUserDefaults standardUserDefaults] stringForKey:@"AlertWhenDone"] isEqualToString: @"Shut Down Computer"]) 
-                { 
-               /* Shut Down */ 
-               NSDictionary* errorDict; 
-               NSAppleEventDescriptor* returnDescriptor = NULL; 
-               NSAppleScript* scriptObject = [[NSAppleScript alloc] initWithSource: 
-                        @"tell application \"Finder\" to shut down"]; 
-               returnDescriptor = [scriptObject executeAndReturnError: &errorDict]; 
-               [scriptObject release]; 
-               [self enableUI: YES]; 
+
+                /* If sleep has been selected */
+                if ([[[NSUserDefaults standardUserDefaults] stringForKey:@"AlertWhenDone"] isEqualToString: @"Put Computer To Sleep"])
+                {
+                    /* Sleep */
+                    NSDictionary* errorDict;
+                    NSAppleEventDescriptor* returnDescriptor = nil;
+                    NSAppleScript* scriptObject = [[NSAppleScript alloc] initWithSource:
+                            @"tell application \"Finder\" to sleep"];
+                    returnDescriptor = [scriptObject executeAndReturnError: &errorDict];
+                    [scriptObject release];
+                }
+                /* If Shutdown has been selected */
+                if ([[[NSUserDefaults standardUserDefaults] stringForKey:@"AlertWhenDone"] isEqualToString: @"Shut Down Computer"])
+                {
+                    /* Shut Down */
+                    NSDictionary* errorDict;
+                    NSAppleEventDescriptor* returnDescriptor = nil;
+                    NSAppleScript* scriptObject = [[NSAppleScript alloc] initWithSource:
+                            @"tell application \"Finder\" to shut down"];
+                    returnDescriptor = [scriptObject executeAndReturnError: &errorDict];
+                    [scriptObject release];
                 }
-                       
             }
-                       else
-                       {
-                               [self enableUI: YES];
-                       }
             break;
         }
     }
@@ -693,9 +665,9 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
     /* Lets show the queue status here in the main window */
        int queue_count = [fQueueController pendingCount];
        if( queue_count == 1)
-               [fQueueStatus setStringValue: _( @"1 encode queued") ];
+               [fQueueStatus setStringValue: NSLocalizedString( @"1 encode queued", @"" ) ];
     else if (queue_count > 1)
-               [fQueueStatus setStringValue: [NSString stringWithFormat: _( @"%d encodes queued" ), queue_count]];
+               [fQueueStatus setStringValue: [NSString stringWithFormat: NSLocalizedString( @"%d encodes queued", @"" ), queue_count]];
        else
                [fQueueStatus setStringValue: @""];
 }
@@ -724,21 +696,21 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
 // ============================================================
 
 - (void) setupToolbar {
-    toolbar = [[[NSToolbar alloc] initWithIdentifier: @"HandBrake Toolbar"] autorelease];
-    
+    NSToolbar *toolbar = [[[NSToolbar alloc] initWithIdentifier: @"HandBrake Toolbar"] autorelease];
+
     [toolbar setAllowsUserCustomization: YES];
     [toolbar setAutosavesConfiguration: YES];
     [toolbar setDisplayMode: NSToolbarDisplayModeIconAndLabel];
-    
+
     [toolbar setDelegate: self];
-    
+
     [fWindow setToolbar: toolbar];
 }
 
 - (NSToolbarItem *) toolbar: (NSToolbar *)toolbar itemForItemIdentifier:
     (NSString *) itemIdent willBeInsertedIntoToolbar:(BOOL) willBeInserted {
-    NSToolbarItem * item = [[NSToolbarItem alloc] initWithItemIdentifier: itemIdent];
-    
+    NSToolbarItem * item = [[[NSToolbarItem alloc] initWithItemIdentifier: itemIdent] autorelease];
+
     if ([itemIdent isEqualToString: ToggleDrawerIdentifier])
     {
         [item setLabel: @"Toggle Presets"];
@@ -806,7 +778,6 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
     }
     else
     {
-        [item release];
         return nil;
     }
 
@@ -946,6 +917,7 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
                 return NO;
         }
         if (action == @selector(Rip:))
+        {
             if (s.state == HB_STATE_WORKING || s.state == HB_STATE_MUXING || s.state == HB_STATE_PAUSED)
             {
                 if(![[menuItem title] isEqualToString:@"Stop Encoding"])
@@ -961,7 +933,12 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
             else
                 return NO;
         }
-    
+    }
+    if( action == @selector(setDefaultPreset:) )
+    {
+        return [fPresetsOutlineView selectedRow] != -1;
+    }
+
     return YES;
 }
 
@@ -1073,28 +1050,28 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
              * purposes in the title panel
              */
             /* Full Path */
-            [fScanSrcTitlePathField setStringValue: [NSString stringWithFormat:@"%@", scanPath]];
+            [fScanSrcTitlePathField setStringValue:scanPath];
             NSString *displayTitlescanSourceName;
-            
+
             if ([[scanPath lastPathComponent] isEqualToString: @"VIDEO_TS"])
             {
-                /* If VIDEO_TS Folder is chosen, choose its parent folder for the source display name 
+                /* If VIDEO_TS Folder is chosen, choose its parent folder for the source display name
                  we have to use the title->dvd value so we get the proper name of the volume if a physical dvd is the source*/
-                displayTitlescanSourceName = [NSString stringWithFormat:[[scanPath stringByDeletingLastPathComponent] lastPathComponent]];
+                displayTitlescanSourceName = [[scanPath stringByDeletingLastPathComponent] lastPathComponent];
             }
             else
             {
                 /* if not the VIDEO_TS Folder, we can assume the chosen folder is the source name */
-                displayTitlescanSourceName = [NSString stringWithFormat:[scanPath lastPathComponent]];
+                displayTitlescanSourceName = [scanPath lastPathComponent];
             }
             /* we set the source display name in the title selection dialogue */
-            [fSrcDsplyNameTitleScan setStringValue: [NSString stringWithFormat:@"%@", displayTitlescanSourceName]];
+            [fSrcDsplyNameTitleScan setStringValue:displayTitlescanSourceName];
             /* we set the attempted scans display name for main window to displayTitlescanSourceName*/
             browsedSourceDisplayName = [displayTitlescanSourceName retain];
-            /* We show the actual sheet where the user specifies the title to be scanned 
+            /* We show the actual sheet where the user specifies the title to be scanned
              * as we are going to do a title specific scan
              */
-            [self showSourceTitleScanPanel:NULL];
+            [self showSourceTitleScanPanel:nil];
         }
         else
         {
@@ -1111,7 +1088,7 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
                     [self writeToActivityLog:"trying to open eyetv package"];
                     /* We're looking at an EyeTV package - try to open its enclosed
                      .mpg media file */
-                     browsedSourceDisplayName = [[NSString stringWithFormat:@"%@",[[path stringByDeletingPathExtension] lastPathComponent]] retain];
+                     browsedSourceDisplayName = [[[path stringByDeletingPathExtension] lastPathComponent] retain];
                     NSString *mpgname;
                     int n = [[path stringByAppendingString: @"/"]
                              completePathIntoString: &mpgname caseSensitive: NO
@@ -1135,41 +1112,42 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
                 else if ([[path pathExtension] isEqualToString: @"dvdmedia"])
                 {
                     /* path IS a package - but dvdmedia packages can be treaded like normal directories */
-                    browsedSourceDisplayName = [[NSString stringWithFormat:@"%@",[[path stringByDeletingPathExtension] lastPathComponent]] retain];
+                    browsedSourceDisplayName = [[[path stringByDeletingPathExtension] lastPathComponent] retain];
                     [self writeToActivityLog:"trying to open dvdmedia package"];
                     [self performScan:path scanTitleNum:0];
                 }
-                else 
+                else
                 {
                     /* The package is not an eyetv package, so we do not call performScan */
                     [self writeToActivityLog:"unable to open package"];
                 }
             }
-            else // path is not a package, so we treat it as a dvd parent folder or VIDEO_TS folder 
+            else // path is not a package, so we treat it as a dvd parent folder or VIDEO_TS folder
             {
                 /* path is not a package, so we call perform scan directly on our file */
                 if ([[path lastPathComponent] isEqualToString: @"VIDEO_TS"])
                 {
                     [self writeToActivityLog:"trying to open video_ts folder (video_ts folder chosen)"];
                     /* If VIDEO_TS Folder is chosen, choose its parent folder for the source display name*/
-                    browsedSourceDisplayName = [[NSString stringWithFormat:@"%@",[[path stringByDeletingLastPathComponent] lastPathComponent]] retain];
+                    browsedSourceDisplayName = [[[path stringByDeletingLastPathComponent] lastPathComponent] retain];
                 }
                 else
                 {
                     [self writeToActivityLog:"trying to open video_ts folder (parent directory chosen)"];
                     /* if not the VIDEO_TS Folder, we can assume the chosen folder is the source name */
-                    browsedSourceDisplayName = [[NSString stringWithFormat:@"%@",[path lastPathComponent]] retain];
+                    /* make sure we remove any path extension as this can also be an '.mpg' file */
+                    browsedSourceDisplayName = [[path lastPathComponent] retain];
                 }
                 [self performScan:path scanTitleNum:0];
             }
-            
+
         }
-        
+
     }
     else // User clicked Cancel in browse window
     {
         /* if we have a title loaded up */
-        if ([[fSrcDVD2Field stringValue] length] > 0)
+        if ([[fSrcDVD2Field stringValue] length] > 0 && SuccessfulScan)
         {
             [self enableUI: YES];
         }
@@ -1184,16 +1162,14 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
     */
     [fScanSrcTitleNumField setStringValue: @"0"];
        /* Show the panel */
-       [NSApp beginSheet: fScanSrcTitlePanel modalForWindow: fWindow modalDelegate: NULL didEndSelector: NULL contextInfo: NULL];
+       [NSApp beginSheet:fScanSrcTitlePanel modalForWindow:fWindow modalDelegate:nil didEndSelector:NULL contextInfo:NULL];
 }
 
 - (IBAction) closeSourceTitleScanPanel: (id) sender
 {
     [NSApp endSheet: fScanSrcTitlePanel];
     [fScanSrcTitlePanel orderOut: self];
-    
-    
-    
+
     if(sender == fScanSrcTitleOpenButton)
     {
         /* We setup the scan status in the main window to indicate a source title scan */
@@ -1209,7 +1185,6 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
     }
 }
 
-
 /* Here we actually tell hb_scan to perform the source scan, using the path to source and title number*/
 - (void) performScan:(NSString *) scanPath scanTitleNum: (int) scanTitleNum
 {
@@ -1217,13 +1192,18 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
     BOOL cancelScanDecrypt = 0;
     NSString *path = scanPath;
     HBDVDDetector *detector = [HBDVDDetector detectorForPath:path];
+
+    // Notify ChapterTitles that there's no title
+    [fChapterTitlesDelegate resetWithTitle:nil];
+    [fChapterTable reloadData];
+
     if( [detector isVideoDVD] )
     {
         // The chosen path was actually on a DVD, so use the raw block
         // device path instead.
         path = [detector devicePath];
         [self writeToActivityLog: "trying to open a physical dvd at: %s", [scanPath UTF8String]];
-        
+
         /* lets check for vlc here to make sure we have a dylib available to use for decrypting */
         NSString *vlcPath = @"/Applications/VLC.app";
         NSFileManager * fileManager = [NSFileManager defaultManager];
@@ -1252,21 +1232,17 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
             cancelScanDecrypt = 0;
             [self writeToActivityLog: "user overrode vlc warning -trying to open physical dvd without decryption"];
             }
-            
+
         }
         else
         {
             /* VLC was found in /Applications so all is well, we can carry on using vlc's libdvdcss.dylib for decrypting if needed */
             [self writeToActivityLog: "VLC app found for decrypting physical dvd"];
         }
-        
     }
-    
-    
-    
+
     if (cancelScanDecrypt == 0)
     {
-        
         /* we actually pass the scan off to libhb here */
         /* If there is no title number passed to scan, we use "0"
          * which causes the default behavior of a full source scan
@@ -1279,19 +1255,18 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
         {
             [self writeToActivityLog: "scanning specifically for title: %d", scanTitleNum];
         }
+
         hb_scan( fHandle, [path UTF8String], scanTitleNum );
-        [fSrcDVD2Field setStringValue: [NSString stringWithFormat: @"Scanning new source ..."]];
+        [fSrcDVD2Field setStringValue:@"Scanning new source ..."];
     }
     else
     {
             /* if we have a title loaded up */
-        if ([[fSrcDVD2Field stringValue] length] > 0)
+        if ([[fSrcDVD2Field stringValue] length] > 0 && SuccessfulScan)
         {
             [self enableUI: YES];
         }
     }
-
-    
 }
 
 - (IBAction) showNewScan:(id)sender
@@ -1300,15 +1275,15 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
        hb_title_t * title;
        int indxpri=0;    // Used to search the longuest title (default in combobox)
        int longuestpri=0; // Used to search the longuest title (default in combobox)
-       
+
        list = hb_get_titles( fHandle );
-       
+
        if( !hb_list_count( list ) )
        {
                /* We display a message if a valid dvd source was not chosen */
                [fSrcDVD2Field setStringValue: @"No Valid Source Found"];
         SuccessfulScan = NO;
-        
+
         // Notify ChapterTitles that there's no title
         [fChapterTitlesDelegate resetWithTitle:nil];
         [fChapterTable reloadData];
@@ -1319,33 +1294,33 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
         which we use at the end of this function to tell the gui
         if this is the first successful scan since launch and whether
         or not we should set all settings to the defaults */
-               
+
         currentSuccessfulScanCount++;
-        
-        [toolbar validateVisibleItems];
-               
+
+        [[fWindow toolbar] validateVisibleItems];
+
                [fSrcTitlePopUp removeAllItems];
                for( int i = 0; i < hb_list_count( list ); i++ )
                {
                        title = (hb_title_t *) hb_list_item( list, i );
-                       
+
             currentSource = [NSString stringWithUTF8String: title->name];
-            
+
             /*Set DVD Name at top of window with the browsedSourceDisplayName grokked right before -performScan */
-                       [fSrcDVD2Field setStringValue: [NSString stringWithFormat: @"%@",browsedSourceDisplayName]];
-                       
-                       /* Use the dvd name in the default output field here 
+                       [fSrcDVD2Field setStringValue:browsedSourceDisplayName];
+
+                       /* Use the dvd name in the default output field here
                                May want to add code to remove blank spaces for some dvd names*/
                        /* Check to see if the last destination has been set,use if so, if not, use Desktop */
-                       if ([[NSUserDefaults standardUserDefaults] stringForKey:@"LastDestinationDirectory"])
+            if ([[NSUserDefaults standardUserDefaults] stringForKey:@"LastDestinationDirectory"])
                        {
                                [fDstFile2Field setStringValue: [NSString stringWithFormat:
-                                       @"%@/%@.mp4", [[NSUserDefaults standardUserDefaults] stringForKey:@"LastDestinationDirectory"],browsedSourceDisplayName]];
+                                       @"%@/%@.mp4", [[NSUserDefaults standardUserDefaults] stringForKey:@"LastDestinationDirectory"],[browsedSourceDisplayName stringByDeletingPathExtension]]];
                        }
                        else
                        {
                                [fDstFile2Field setStringValue: [NSString stringWithFormat:
-                                       @"%@/Desktop/%@.mp4", NSHomeDirectory(),browsedSourceDisplayName]];
+                                       @"%@/Desktop/%@.mp4", NSHomeDirectory(),[browsedSourceDisplayName stringByDeletingPathExtension]]];
                        }
                        
                        if (longuestpri < title->hours*60*60 + title->minutes *60 + title->seconds)
@@ -1363,18 +1338,18 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
                 title->index, title->hours, title->minutes,
                 title->seconds]];
                }
-        
+
                // Select the longuest title
                [fSrcTitlePopUp selectItemAtIndex: indxpri];
-               [self titlePopUpChanged: NULL];
-               
+               [self titlePopUpChanged:nil];
+
         SuccessfulScan = YES;
                [self enableUI: YES];
-               
+
                /* if its the initial successful scan after awakeFromNib */
         if (currentSuccessfulScanCount == 1)
         {
-            [self selectDefaultPreset: NULL];
+            [self selectDefaultPreset:nil];
             /* if Deinterlace upon launch is specified in the prefs, then set to 1 for "Fast",
              if not, then set to 0 for none */
             if ([[NSUserDefaults standardUserDefaults] boolForKey:@"DefaultDeinterlaceOn"] > 0)
@@ -1465,12 +1440,12 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
 
 
     /* If mpeg-4, then set mpeg-4 specific options like chapters and > 4gb file sizes */
-       if ([fDstFormatPopUp indexOfSelectedItem] == 0)
+       if( [fDstFormatPopUp indexOfSelectedItem] == 0 )
        {
         /* We set the largeFileSize (64 bit formatting) variable here to allow for > 4gb files based on the format being
                mpeg4 and the checkbox being checked 
                *Note: this will break compatibility with some target devices like iPod, etc.!!!!*/
-               if ([fDstMp4LargeFileCheck state] == NSOnState)
+               if( [fDstMp4LargeFileCheck state] == NSOnState )
                {
                        job->largeFileSize = 1;
                }
@@ -1479,7 +1454,7 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
                        job->largeFileSize = 0;
                }
         /* We set http optimized mp4 here */
-        if ([fDstMp4HttpOptFileCheck state] == NSOnState)
+        if( [fDstMp4HttpOptFileCheck state] == NSOnState && [fDstMp4HttpOptFileCheck isEnabled] )
                {
         job->mp4_optimize = 1;
         }
@@ -1488,7 +1463,7 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
         job->mp4_optimize = 0;
         }
     }
-       if ([fDstFormatPopUp indexOfSelectedItem] == 0 || [fDstFormatPopUp indexOfSelectedItem] == 1)
+       if( [fDstFormatPopUp indexOfSelectedItem] == 0 || [fDstFormatPopUp indexOfSelectedItem] == 1 )
        {
          /* We set the chapter marker extraction here based on the format being
                mpeg4 or mkv and the checkbox being checked */
@@ -1501,14 +1476,7 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
                        job->chapter_markers = 0;
                }
        }
-       /* This likely is no longer needed as we are directly setting the job->vcodec */
-    /*
-    if( ( job->vcodec & HB_VCODEC_FFMPEG ) &&
-        [fVidEncoderPopUp indexOfSelectedItem] > 0 )
-    {
-        job->vcodec = HB_VCODEC_XVID;
-    }
-    */
+       
     if( job->vcodec & HB_VCODEC_X264 )
     {
                if ([fDstMp4iPodFileCheck state] == NSOnState)
@@ -1553,11 +1521,18 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
         job->vrate      = 27000000;
         job->vrate_base = hb_video_rates[[fVidRatePopUp
             indexOfSelectedItem]-1].rate;
+        /* We are not same as source so we set job->cfr to 1 
+         * to enable constant frame rate since user has specified
+         * a specific framerate*/
+        job->cfr = 1;
     }
     else
     {
         job->vrate      = title->rate;
         job->vrate_base = title->rate_base;
+        /* We are same as source so we set job->cfr to 0 
+         * to enable true same as source framerate */
+        job->cfr = 0;
     }
 
     switch( [fVidQualityMatrix selectedRow] )
@@ -1600,7 +1575,7 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
         audio->out.codec = [[fAudTrack1CodecPopUp selectedItem] tag];
         audio->out.mixdown = [[fAudTrack1MixPopUp selectedItem] tag];
         audio->out.bitrate = [[fAudTrack1BitratePopUp selectedItem] tag];
-        audio->out.samplerate = hb_audio_rates[[fAudTrack1RatePopUp indexOfSelectedItem]].rate;
+        audio->out.samplerate = [[fAudTrack1RatePopUp selectedItem] tag];
         audio->out.dynamic_range_compression = [fAudTrack1DrcField floatValue];
         
         hb_audio_add( job, audio );
@@ -1616,7 +1591,7 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
         audio->out.codec = [[fAudTrack2CodecPopUp selectedItem] tag];
         audio->out.mixdown = [[fAudTrack2MixPopUp selectedItem] tag];
         audio->out.bitrate = [[fAudTrack2BitratePopUp selectedItem] tag];
-        audio->out.samplerate = hb_audio_rates[[fAudTrack2RatePopUp indexOfSelectedItem]].rate;
+        audio->out.samplerate = [[fAudTrack2RatePopUp selectedItem] tag];
         audio->out.dynamic_range_compression = [fAudTrack2DrcField floatValue];
         
         hb_audio_add( job, audio );
@@ -1634,7 +1609,7 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
         audio->out.codec = [[fAudTrack3CodecPopUp selectedItem] tag];
         audio->out.mixdown = [[fAudTrack3MixPopUp selectedItem] tag];
         audio->out.bitrate = [[fAudTrack3BitratePopUp selectedItem] tag];
-        audio->out.samplerate = hb_audio_rates[[fAudTrack3RatePopUp indexOfSelectedItem]].rate;
+        audio->out.samplerate = [[fAudTrack3RatePopUp selectedItem] tag];
         audio->out.dynamic_range_compression = [fAudTrack3DrcField floatValue];
         
         hb_audio_add( job, audio );
@@ -1652,7 +1627,7 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
         audio->out.codec = [[fAudTrack4CodecPopUp selectedItem] tag];
         audio->out.mixdown = [[fAudTrack4MixPopUp selectedItem] tag];
         audio->out.bitrate = [[fAudTrack4BitratePopUp selectedItem] tag];
-        audio->out.samplerate = hb_audio_rates[[fAudTrack4RatePopUp indexOfSelectedItem]].rate;
+        audio->out.samplerate = [[fAudTrack4RatePopUp selectedItem] tag];
         audio->out.dynamic_range_compression = [fAudTrack4DrcField floatValue];
         
         hb_audio_add( job, audio );
@@ -1672,13 +1647,28 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
     
     /* Filters */ 
     job->filters = hb_list_init();
-   
+    
+    /* Now lets call the filters if applicable.
+    * The order of the filters is critical
+    */
+    
        /* Detelecine */
     if ([fPictureController detelecine])
     {
         hb_list_add( job->filters, &hb_filter_detelecine );
     }
-   
+    
+    /* Decomb */
+    if ([fPictureController decomb] > 0)
+    {
+        /* Run old deinterlacer fd by default */
+        //fPicSettingDecomb
+        hb_filter_decomb.settings = (char *) [[fPicSettingDecomb stringValue] UTF8String];
+        //hb_filter_decomb.settings = "4:10:15:9:10:35:9"; // <-- jbrjakes recommended parameters as of 5/23/08
+        hb_list_add( job->filters, &hb_filter_decomb );
+    }
+
+    
     /* Deinterlace */
     if ([fPictureController deinterlace] == 1)
     {
@@ -1699,8 +1689,7 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
         hb_list_add( job->filters, &hb_filter_deinterlace );            
     }
        
-       /* Denoise */
-       
+    /* Denoise */
        if ([fPictureController denoise] == 1) // Weak in popup
        {
                hb_filter_denoise.settings = "2:1:2:3"; 
@@ -1744,24 +1733,24 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
        if( [[NSFileManager defaultManager] fileExistsAtPath:
             [fDstFile2Field stringValue]] )
     {
-        NSBeginCriticalAlertSheet( _( @"File already exists" ),
-            _( @"Cancel" ), _( @"Overwrite" ), NULL, fWindow, self,
+        NSBeginCriticalAlertSheet( NSLocalizedString( @"File already exists", @"" ),
+            NSLocalizedString( @"Cancel", @"" ), NSLocalizedString( @"Overwrite", @"" ), nil, fWindow, self,
             @selector( overwriteAddToQueueAlertDone:returnCode:contextInfo: ),
             NULL, NULL, [NSString stringWithFormat:
-            _( @"Do you want to overwrite %@?" ),
+            NSLocalizedString( @"Do you want to overwrite %@?", @"" ),
             [fDstFile2Field stringValue]] );
         // overwriteAddToQueueAlertDone: will be called when the alert is dismissed.
     }
-    
+
     // Warn if another pending job in the queue has the same destination path
     else if ( ([fQueueController pendingJobGroupWithDestinationPath:[fDstFile2Field stringValue]] != nil)
             || ([[[fQueueController currentJobGroup] destinationPath] isEqualToString: [fDstFile2Field stringValue]]) )
     {
-        NSBeginCriticalAlertSheet( _( @"Another queued encode has specified the same destination." ),
-            _( @"Cancel" ), _( @"Overwrite" ), NULL, fWindow, self,
+        NSBeginCriticalAlertSheet( NSLocalizedString( @"Another queued encode has specified the same destination.", @"" ),
+            NSLocalizedString( @"Cancel", @"" ), NSLocalizedString( @"Overwrite", @"" ), nil, fWindow, self,
             @selector( overwriteAddToQueueAlertDone:returnCode:contextInfo: ),
             NULL, NULL, [NSString stringWithFormat:
-            _( @"Do you want to overwrite %@?" ),
+            NSLocalizedString( @"Do you want to overwrite %@?", @"" ),
             [fDstFile2Field stringValue]] );
         // overwriteAddToQueueAlertDone: will be called when the alert is dismissed.
     }
@@ -1931,18 +1920,18 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
     /* We check for duplicate name here */
     if( [[NSFileManager defaultManager] fileExistsAtPath:[fDstFile2Field stringValue]] )
     {
-        NSBeginCriticalAlertSheet( _( @"File already exists" ),
-            _( @"Cancel" ), _( @"Overwrite" ), NULL, fWindow, self,
+        NSBeginCriticalAlertSheet( NSLocalizedString( @"File already exists", @"" ),
+            NSLocalizedString( @"Cancel", "" ), NSLocalizedString( @"Overwrite", @"" ), nil, fWindow, self,
             @selector( overWriteAlertDone:returnCode:contextInfo: ),
             NULL, NULL, [NSString stringWithFormat:
-            _( @"Do you want to overwrite %@?" ),
+            NSLocalizedString( @"Do you want to overwrite %@?", @"" ),
             [fDstFile2Field stringValue]] );
-            
+
         // overWriteAlertDone: will be called when the alert is dismissed. It will call doRip.
     }
     else
     {
-        /* if there are no jobs in the queue, then add this one to the queue and rip 
+        /* if there are no jobs in the queue, then add this one to the queue and rip
         otherwise, just rip the queue */
         if( hb_count( fHandle ) == 0)
         {
@@ -1985,11 +1974,11 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
                NSBeep();
                reminduser = NSRunAlertPanel(@"The computer will sleep after encoding is done.",@"You have selected to sleep the computer after encoding. To turn off sleeping, go to the HandBrake preferences.", @"OK", @"Preferences...", nil);
                [NSApp requestUserAttention:NSCriticalRequest];
-               if ( reminduser == NSAlertAlternateReturn ) 
+               if ( reminduser == NSAlertAlternateReturn )
                {
-                       [self showPreferencesWindow:NULL];
+                       [self showPreferencesWindow:nil];
                }
-       } 
+       }
        else if ([[[NSUserDefaults standardUserDefaults] stringForKey:@"AlertWhenDone"] isEqualToString: @"Shut Down Computer"])
        {
                /*Warn that computer will shut down after encoding*/
@@ -1997,9 +1986,9 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
                NSBeep();
                reminduser = NSRunAlertPanel(@"The computer will shut down after encoding is done.",@"You have selected to shut down the computer after encoding. To turn off shut down, go to the HandBrake preferences.", @"OK", @"Preferences...", nil);
                [NSApp requestUserAttention:NSCriticalRequest];
-               if ( reminduser == NSAlertAlternateReturn ) 
+               if ( reminduser == NSAlertAlternateReturn )
                {
-                       [self showPreferencesWindow:NULL];
+                       [self showPreferencesWindow:nil];
                }
        }
 
@@ -2085,10 +2074,6 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
         [self doCancelCurrentJob];  // <- this also stops libhb
 }
 
-
-
-
-
 - (IBAction) Pause: (id) sender
 {
     hb_state_t s;
@@ -2112,15 +2097,15 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
     hb_list_t  * list  = hb_get_titles( fHandle );
     hb_title_t * title = (hb_title_t*)
         hb_list_item( list, [fSrcTitlePopUp indexOfSelectedItem] );
-               
-               
+
+
     /* If Auto Naming is on. We create an output filename of dvd name - title number */
-    if ([[NSUserDefaults standardUserDefaults] boolForKey:@"DefaultAutoNaming"] > 0)
+    if( [[NSUserDefaults standardUserDefaults] boolForKey:@"DefaultAutoNaming"] > 0 && ( hb_list_count( list ) > 1 ) )
        {
                [fDstFile2Field setStringValue: [NSString stringWithFormat:
                        @"%@/%@-%d.%@", [[fDstFile2Field stringValue] stringByDeletingLastPathComponent],
-                       browsedSourceDisplayName,
-                         title->index,
+                       [browsedSourceDisplayName stringByDeletingPathExtension],
+            title->index,
                        [[fDstFile2Field stringValue] pathExtension]]]; 
        }
 
@@ -2137,11 +2122,11 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
     [fSrcChapterStartPopUp selectItemAtIndex: 0];
     [fSrcChapterEndPopUp   selectItemAtIndex:
         hb_list_count( title->list_chapter ) - 1];
-    [self chapterPopUpChanged: NULL];
+    [self chapterPopUpChanged:nil];
 
 /* Start Get and set the initial pic size for display */
        hb_job_t * job = title->job;
-       fTitle = title; 
+       fTitle = title;
 
        /* Pixel Ratio Setting */
        if ([[NSUserDefaults standardUserDefaults] boolForKey:@"PixelRatio"])
@@ -2187,13 +2172,13 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
             subtitle->lang] action: NULL keyEquivalent: @""];
     }
     [fSubPopUp selectItemAtIndex: 0];
-       
-       [self subtitleSelectionChanged: NULL];
-    
+
+       [self subtitleSelectionChanged:nil];
+
     /* Update chapter table */
     [fChapterTitlesDelegate resetWithTitle:title];
     [fChapterTable reloadData];
-   
+
    /* Lets make sure there arent any erroneous audio tracks in the job list, so lets make sure its empty*/
     int audiotrack_count = hb_list_count(job->list_audio);
     for( int i = 0; i < audiotrack_count;i++)
@@ -2201,7 +2186,7 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
         hb_audio_t * temp_audio = (hb_audio_t*) hb_list_item( job->list_audio, 0 );
         hb_list_rem(job->list_audio, temp_audio);
     }
-   
+
     /* Update audio popups */
     [self addAllAudioTracksToPopUp: fAudLang1PopUp];
     [self addAllAudioTracksToPopUp: fAudLang2PopUp];
@@ -2210,10 +2195,10 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
     /* search for the first instance of our prefs default language for track 1, and set track 2 to "none" */
        NSString * audioSearchPrefix = [[NSUserDefaults standardUserDefaults] stringForKey:@"DefaultLanguage"];
         [self selectAudioTrackInPopUp: fAudLang1PopUp searchPrefixString: audioSearchPrefix selectIndexIfNotFound: 1];
-    [self selectAudioTrackInPopUp: fAudLang2PopUp searchPrefixString: NULL selectIndexIfNotFound: 0];
-    [self selectAudioTrackInPopUp: fAudLang3PopUp searchPrefixString: NULL selectIndexIfNotFound: 0];
-    [self selectAudioTrackInPopUp: fAudLang4PopUp searchPrefixString: NULL selectIndexIfNotFound: 0];
-       
+    [self selectAudioTrackInPopUp:fAudLang2PopUp searchPrefixString:nil selectIndexIfNotFound:0];
+    [self selectAudioTrackInPopUp:fAudLang3PopUp searchPrefixString:nil selectIndexIfNotFound:0];
+    [self selectAudioTrackInPopUp:fAudLang4PopUp searchPrefixString:nil selectIndexIfNotFound:0];
+
        /* changing the title may have changed the audio channels on offer, */
        /* so call audioTrackPopUpChanged for both audio tracks to update the mixdown popups */
        [self audioTrackPopUpChanged: fAudLang1PopUp];
@@ -2221,25 +2206,12 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
     [self audioTrackPopUpChanged: fAudLang3PopUp];
     [self audioTrackPopUpChanged: fAudLang4PopUp];
     
-    /* We repopulate the Video Framerate popup and show the detected framerate along with "Same as Source"*/
+    /* We repopulate the Video Framerate popup */
+
     [fVidRatePopUp removeAllItems];
-    if (fTitle->rate_base == 1126125) // 23.976 NTSC Film
-    {
-        [fVidRatePopUp addItemWithTitle: @"Same as source (23.976)"];
-    }
-    else if (fTitle->rate_base == 1080000) // 25 PAL Film/Video
-    {
-        [fVidRatePopUp addItemWithTitle: @"Same as source (25)"];
-    }
-    else if (fTitle->rate_base == 900900) // 29.97 NTSC Video
-    {
-        [fVidRatePopUp addItemWithTitle: @"Same as source (29.97)"];
-    }
-    else
-    {
-        /* if none of the common dvd source framerates is detected, just use "Same as source" */
-        [fVidRatePopUp addItemWithTitle: @"Same as source"];
-    }
+    
+    [fVidRatePopUp addItemWithTitle:@"Same as source"];
+    
        for( int i = 0; i < hb_video_rates_count; i++ )
     {
         if ([[NSString stringWithCString: hb_video_rates[i].string] isEqualToString: [NSString stringWithFormat: @"%.3f",23.976]])
@@ -2262,20 +2234,19 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
                        [fVidRatePopUp addItemWithTitle:
                                [NSString stringWithCString: hb_video_rates[i].string]];
                }
-    }   
+    }
     [fVidRatePopUp selectItemAtIndex: 0];
-    
+
     /* we run the picture size values through calculatePictureSizing to get all picture setting        information*/
-       [self calculatePictureSizing: NULL];
-    
+       [self calculatePictureSizing:nil];
+
    /* lets call tableViewSelected to make sure that any preset we have selected is enforced after a title change */
-       [self selectPreset:NULL]; 
-       
+       [self selectPreset:nil];
 }
 
 - (IBAction) chapterPopUpChanged: (id) sender
 {
-    
+
        /* If start chapter popup is greater than end chapter popup,
        we set the end chapter popup to the same as start chapter popup */
        if ([fSrcChapterStartPopUp indexOfSelectedItem] > [fSrcChapterEndPopUp indexOfSelectedItem])
@@ -2379,91 +2350,89 @@ static NSString *        ChooseSourceIdentifier             = @"Choose Source It
                        break;
     }
     [fVidEncoderPopUp selectItemAtIndex: 0];
-    
+
     [self audioAddAudioTrackCodecs: fAudTrack1CodecPopUp];
     [self audioAddAudioTrackCodecs: fAudTrack2CodecPopUp];
     [self audioAddAudioTrackCodecs: fAudTrack3CodecPopUp];
     [self audioAddAudioTrackCodecs: fAudTrack4CodecPopUp];
-    
-    /* FIX ME: we need to restore changing the file extension as pwer */
-    
-    if( [string characterAtIndex: [string length] - 4] == '.' )
-        {
-            [fDstFile2Field setStringValue: [NSString stringWithFormat:
-                                             @"%@.%s", [string substringToIndex: [string length] - 4],
-                                             ext]];
-        }
-        else
-        {
-            [fDstFile2Field setStringValue: [NSString stringWithFormat:
-                                             @"%@.%s", string, ext]];
-        }
-        
-    if ( SuccessfulScan ) 
+
+    if( format == 0 )
+        [self autoSetM4vExtension: sender];
+    else
+        [fDstFile2Field setStringValue: [NSString stringWithFormat:@"%@.%s", [string stringByDeletingPathExtension], ext]];
+
+    if( SuccessfulScan )
     {
         /* Add/replace to the correct extension */
         [self audioTrackPopUpChanged: fAudLang1PopUp];
         [self audioTrackPopUpChanged: fAudLang2PopUp];
         [self audioTrackPopUpChanged: fAudLang3PopUp];
         [self audioTrackPopUpChanged: fAudLang4PopUp];
-        
-        if ( [fVidEncoderPopUp selectedItem] == NULL )
+
+        if( [fVidEncoderPopUp selectedItem] == nil )
         {
-            
+
             [fVidEncoderPopUp selectItemAtIndex:0];
-            [self videoEncoderPopUpChanged: NULL];
-            
+            [self videoEncoderPopUpChanged:nil];
+
             /* changing the format may mean that we can / can't offer mono or 6ch, */
             /* so call audioTrackPopUpChanged for both audio tracks to update the mixdown popups */
-            
-            
-            
+
             /* We call the method to properly enable/disable turbo 2 pass */
             [self twoPassCheckboxChanged: sender];
             /* We call method method to change UI to reflect whether a preset is used or not*/
         }
     }
-    
-    /* Lets check to see if we want to auto set the .m4v extension for mp4 */
-    [self autoSetM4vExtension: sender];
-       [self customSettingUsed: sender];       
+       [self customSettingUsed: sender];
 }
 
+- (IBAction) autoSetM4vExtension: (id) sender
+{
+    if ( [fDstFormatPopUp indexOfSelectedItem] )
+        return;
 
+    NSString * extension = @"mp4";
 
-    /* if MP4 format and [fDstCodecsPopUp indexOfSelectedItem] > 1 we know that the audio is going to be
-         * either aac + ac3 passthru, or just ac3 passthru so we need to make sure the output file extension is m4v
-         * otherwise Quicktime will not play it at all */
-- (IBAction) autoSetM4vExtension: (id) sender
+    if( [[fAudTrack1CodecPopUp selectedItem] tag] == HB_ACODEC_AC3 || [[fAudTrack2CodecPopUp selectedItem] tag] == HB_ACODEC_AC3 ||
+                                                        [[fAudTrack3CodecPopUp selectedItem] tag] == HB_ACODEC_AC3 ||
+                                                        [[fAudTrack4CodecPopUp selectedItem] tag] == HB_ACODEC_AC3 ||
+                                                        [fCreateChapterMarkers state] == NSOnState ||
+                                                        [[NSUserDefaults standardUserDefaults] boolForKey:@"DefaultMpegName"] > 0 )
+    {
+        extension = @"m4v";
+    }
+
+    if( [extension isEqualTo: [[fDstFile2Field stringValue] pathExtension]] )
+        return;
+    else
+        [fDstFile2Field setStringValue: [NSString stringWithFormat:@"%@.%@",
+                                    [[fDstFile2Field stringValue] stringByDeletingPathExtension], extension]];
+}
+
+- (void) shouldEnableHttpMp4CheckBox: (id) sender
 {
-        /*FIX ME: for this to work, we will now have to iterate through the audio list to see if ac3 in an mp4 is chosen 
-        * for now just comment it out.
-        */
-        /*
-        if ([fDstFormatPopUp indexOfSelectedItem] == 0 && [fDstCodecsPopUp indexOfSelectedItem] > 1)
-        {
-            NSString *newpath = [[[fDstFile2Field stringValue] stringByDeletingPathExtension] stringByAppendingPathExtension: @"m4v"];
-            [fDstFile2Field setStringValue: [NSString stringWithFormat:
-                                             @"%@", newpath]];
-        }
-        */
+    if( [[fAudTrack1CodecPopUp selectedItem] tag] == HB_ACODEC_AC3 || [[fAudTrack2CodecPopUp selectedItem] tag] == HB_ACODEC_AC3 ||
+                                                        [[fAudTrack3CodecPopUp selectedItem] tag] == HB_ACODEC_AC3 ||
+                                                        [[fAudTrack4CodecPopUp selectedItem] tag] == HB_ACODEC_AC3 )
+        [fDstMp4HttpOptFileCheck setEnabled: NO];
+    else
+        [fDstMp4HttpOptFileCheck setEnabled: YES];
 }
+        
 /* Method to determine if we should change the UI
 To reflect whether or not a Preset is being used or if
 the user is using "Custom" settings by determining the sender*/
 - (IBAction) customSettingUsed: (id) sender
 {
-       if ([sender stringValue] != NULL)
+       if ([sender stringValue])
        {
                /* Deselect the currently selected Preset if there is one*/
                [fPresetsOutlineView deselectRow:[fPresetsOutlineView selectedRow]];
-               [[fPresetsActionMenu itemAtIndex:0] setEnabled: NO];
                /* Change UI to show "Custom" settings are being used */
                [fPresetSelectedDisplay setStringValue: @"Custom"];
-               
+
                curUserPresetChosenNum = nil;
        }
-
 }
 
 
@@ -2590,14 +2559,14 @@ the user is using "Custom" settings by determining the sender*/
 - (IBAction) qualitySliderChanged: (id) sender
 {
     [fVidConstantCell setTitle: [NSString stringWithFormat:
-        _( @"Constant quality: %.0f %%" ), 100.0 *
+        NSLocalizedString( @"Constant quality: %.0f %%", @"" ), 100.0 *
         [fVidQualitySlider floatValue]]];
                [self customSettingUsed: sender];
 }
 
 - (void) controlTextDidChange: (NSNotification *) notification
 {
-    [self calculateBitrate: NULL];
+    [self calculateBitrate:nil];
 }
 
 - (IBAction) calculateBitrate: (id) sender
@@ -2651,7 +2620,7 @@ the user is using "Custom" settings by determining the sender*/
  */
 
 - (void)pictureSettingsDidChange {
-       [self calculatePictureSizing: NULL];
+       [self calculatePictureSizing:nil];
 }
 
 /* Get and Display Current Pic Settings in main window */
@@ -2677,18 +2646,18 @@ the user is using "Custom" settings by determining the sender*/
         hb_set_anamorphic_size(job, &output_width, &output_height, &output_par_width, &output_par_height);
         int display_width;
         display_width = output_width * output_par_width / output_par_height;
-        
+
         [fPicSettingsOutp setStringValue: [NSString stringWithFormat:@"%d x %d", output_width, output_height]];
         [fPicSettingsAnamorphic setStringValue: [NSString stringWithFormat:@"%d x %d Loose", display_width, output_height]];
-        
+
         fTitle->job->keep_ratio = 0;
     }
        else
        {
-        [fPicSettingsAnamorphic setStringValue: [NSString stringWithFormat:@"Off"]];
+        [fPicSettingsAnamorphic setStringValue:@"Off"];
        }
-    
-       /* Set ON/Off values for the deinterlace/keep aspect ratio according to boolean */      
+
+       /* Set ON/Off values for the deinterlace/keep aspect ratio according to boolean */
        if (fTitle->job->keep_ratio > 0)
        {
                [fPicSettingARkeep setStringValue: @"On"];
@@ -2697,6 +2666,7 @@ the user is using "Custom" settings by determining the sender*/
        {
                [fPicSettingARkeep setStringValue: @"Off"];
        }       
+    
     /* Detelecine */
     if ([fPictureController detelecine]) {
         [fPicSettingDetelecine setStringValue: @"Yes"];
@@ -2705,15 +2675,34 @@ the user is using "Custom" settings by determining the sender*/
         [fPicSettingDetelecine setStringValue: @"No"];
     }
     
+    /* Decomb */
+       if ([fPictureController decomb] == 0)
+       {
+               [fPicSettingDecomb setStringValue: @"Off"];
+       }
+       else if ([fPictureController decomb] == 1)
+       {
+               [fPicSettingDecomb setStringValue: @"1:2:6:9:80:16:16"];
+       }
+    else if ([fPictureController decomb] == 2)
+    {
+        [fPicSettingDecomb setStringValue:[[NSUserDefaults standardUserDefaults] stringForKey:@"DecombCustomString"]];
+    }
+
     /* VFR (Variable Frame Rate) */
     if ([fPictureController vfr]) {
         /* We change the string of the fps popup to warn that vfr is on Framerate (FPS): */
-        [fVidRateField setStringValue: @"Framerate (VFR On):"];  
+        [fVidRateField setStringValue: @"Framerate (VFR On):"]; 
+        /* for VFR we select same as source (or title framerate) and disable the popup.
+        * We know its index 0 as that is determined in titlePopUpChanged */
+        [fVidRatePopUp selectItemAtIndex: 0];
+        [fVidRatePopUp setEnabled: NO];  
         
     }
     else {
         /* make sure the label for framerate is set to its default */  
         [fVidRateField setStringValue: @"Framerate (FPS):"];
+        [fVidRatePopUp setEnabled: YES];
     }
     
        /* Deinterlace */
@@ -2733,7 +2722,8 @@ the user is using "Custom" settings by determining the sender*/
        {
                [fPicSettingDeinterlace setStringValue: @"Slower"];
        }
-       /* Denoise */
+               
+    /* Denoise */
        if ([fPictureController denoise] == 0)
        {
                [fPicSettingDenoise setStringValue: @"Off"];
@@ -2767,7 +2757,8 @@ the user is using "Custom" settings by determining the sender*/
        {
                [fPicSettingPAR setStringValue: @"Off"];
        }
-       /* Set the display field for crop as per boolean */
+       
+    /* Set the display field for crop as per boolean */
        if (![fPictureController autoCrop])
        {
            [fPicSettingAutoCrop setStringValue: @"Custom"];
@@ -2879,6 +2870,13 @@ the user is using "Custom" settings by determining the sender*/
         [fAudTrack1DrcSlider setFloatValue: 1.00];
         [self audioDRCSliderChanged: fAudTrack1DrcSlider];
     }
+    else if ([[fAudTrack1MixPopUp selectedItem] tag] == HB_ACODEC_AC3)
+    {
+        [fAudTrack1RatePopUp setEnabled: NO];
+        [fAudTrack1BitratePopUp setEnabled: NO];
+        [fAudTrack1DrcSlider setEnabled: NO];
+        [fAudTrack1DrcField setEnabled: NO];
+    }
     
     /* enable/disable the mixdown text and popupbutton for audio track 2 */
     [fAudTrack2CodecPopUp setEnabled: ([fAudLang2PopUp indexOfSelectedItem] == 0) ? NO : YES];
@@ -2896,6 +2894,13 @@ the user is using "Custom" settings by determining the sender*/
         [fAudTrack2DrcSlider setFloatValue: 1.00];
         [self audioDRCSliderChanged: fAudTrack2DrcSlider];
     }
+    else if ([[fAudTrack2MixPopUp selectedItem] tag] == HB_ACODEC_AC3)
+    {
+        [fAudTrack2RatePopUp setEnabled: NO];
+        [fAudTrack2BitratePopUp setEnabled: NO];
+        [fAudTrack2DrcSlider setEnabled: NO];
+        [fAudTrack2DrcField setEnabled: NO];
+    }
     
     /* enable/disable the mixdown text and popupbutton for audio track 3 */
     [fAudTrack3CodecPopUp setEnabled: ([fAudLang3PopUp indexOfSelectedItem] == 0) ? NO : YES];
@@ -2913,6 +2918,13 @@ the user is using "Custom" settings by determining the sender*/
         [fAudTrack3DrcSlider setFloatValue: 1.00];
         [self audioDRCSliderChanged: fAudTrack3DrcSlider];
     }
+    else if ([[fAudTrack3MixPopUp selectedItem] tag] == HB_ACODEC_AC3)
+    {
+        [fAudTrack3RatePopUp setEnabled: NO];
+        [fAudTrack3BitratePopUp setEnabled: NO];
+        [fAudTrack3DrcSlider setEnabled: NO];
+        [fAudTrack3DrcField setEnabled: NO];
+    }
     
     /* enable/disable the mixdown text and popupbutton for audio track 4 */
     [fAudTrack4CodecPopUp setEnabled: ([fAudLang4PopUp indexOfSelectedItem] == 0) ? NO : YES];
@@ -2930,6 +2942,13 @@ the user is using "Custom" settings by determining the sender*/
         [fAudTrack4DrcSlider setFloatValue: 1.00];
         [self audioDRCSliderChanged: fAudTrack4DrcSlider];
     }
+    else if ([[fAudTrack4MixPopUp selectedItem] tag] == HB_ACODEC_AC3)
+    {
+        [fAudTrack4RatePopUp setEnabled: NO];
+        [fAudTrack4BitratePopUp setEnabled: NO];
+        [fAudTrack4DrcSlider setEnabled: NO];
+        [fAudTrack4DrcField setEnabled: NO];
+    }
     
 }
 
@@ -2943,7 +2962,7 @@ the user is using "Custom" settings by determining the sender*/
        hb_audio_config_t * audio;
 
     [sender removeAllItems];
-    [sender addItemWithTitle: _( @"None" )];
+    [sender addItemWithTitle: NSLocalizedString( @"None", @"" )];
     for( int i = 0; i < hb_list_count( title->list_audio ); i++ )
     {
         audio = (hb_audio_config_t *) hb_list_audio_config_item( title->list_audio, i );
@@ -2962,8 +2981,8 @@ the user is using "Custom" settings by determining the sender*/
     /* e.g. to find the first French track, pass in an NSString * of "Francais" */
     /* e.g. to find the first English 5.1 AC3 track, pass in an NSString * of "English (AC3) (5.1 ch)" */
     /* if no matching track is found, then selectIndexIfNotFound is used to choose which track to select instead */
-    
-       if (searchPrefixString != NULL) 
+
+       if (searchPrefixString)
        {
 
         for( int i = 0; i < [sender numberOfItems]; i++ )
@@ -3161,42 +3180,43 @@ the user is using "Custom" settings by determining the sender*/
         sampleratePopUp = fAudTrack4RatePopUp;
         bitratePopUp = fAudTrack4BitratePopUp;
     }
-    
+
     /* get the index of the selected audio Track*/
     int thisAudioIndex = [sender indexOfSelectedItem] - 1;
-    
+
     /* pointer for the hb_audio_s struct we will use later on */
     hb_audio_config_t * audio;
-    
+
     int acodec;
     /* check if the audio mixdown controls need their enabled state changing */
-    [self setEnabledStateOfAudioMixdownControls: NULL];
-    
+    [self setEnabledStateOfAudioMixdownControls:nil];
+
     if (thisAudioIndex != -1)
     {
-        
+
         /* get the audio */
         audio = (hb_audio_config_t *) hb_list_audio_config_item( fTitle->list_audio, thisAudioIndex );// Should "fTitle" be title and be setup ?
-        
+
         /* actually manipulate the proper mixdowns here */
         /* delete the previous audio mixdown options */
         [mixdownPopUp removeAllItems];
-        
+
         acodec = [[audiocodecPopUp selectedItem] tag];
-        
+
         if (audio != NULL)
         {
-            
+
             /* find out if our selected output audio codec supports mono and / or 6ch */
             /* we also check for an input codec of AC3 or DCA,
              as they are the only libraries able to do the mixdown to mono / conversion to 6-ch */
             /* audioCodecsSupportMono and audioCodecsSupport6Ch are the same for now,
              but this may change in the future, so they are separated for flexibility */
-            int audioCodecsSupportMono = ((audio->in.codec == HB_ACODEC_AC3 ||
-                                           audio->in.codec == HB_ACODEC_DCA) && acodec == HB_ACODEC_FAAC);
-            int audioCodecsSupport6Ch =  ((audio->in.codec == HB_ACODEC_AC3 ||
-                                           audio->in.codec == HB_ACODEC_DCA) && (acodec == HB_ACODEC_FAAC ||
-                                                                                 acodec == HB_ACODEC_VORBIS));
+            int audioCodecsSupportMono =
+                    (audio->in.codec & (HB_ACODEC_AC3|HB_ACODEC_DCA)) &&
+                    (acodec != HB_ACODEC_LAME);
+            int audioCodecsSupport6Ch =
+                    (audio->in.codec & (HB_ACODEC_AC3|HB_ACODEC_DCA)) &&
+                    (acodec != HB_ACODEC_LAME);
             
             /* check for AC-3 passthru */
             if (audio->in.codec == HB_ACODEC_AC3 && acodec == HB_ACODEC_AC3)
@@ -3342,8 +3362,13 @@ the user is using "Custom" settings by determining the sender*/
         }
     
     }
-
+    if( [fDstFormatPopUp indexOfSelectedItem] == 0 )
+    {
+        [self autoSetM4vExtension: sender];
+        [self shouldEnableHttpMp4CheckBox: sender];
+    }
 }
+
 - (IBAction) audioTrackMixdownChanged: (id) sender
 {
     
@@ -3456,6 +3481,15 @@ the user is using "Custom" settings by determining the sender*/
             maxbitrate = 384;
             
     }
+    
+    /* make sure we have a selected title before continuing */
+    if (fTitle == NULL) return;
+    /* get the audio so we can find out what input rates are*/
+    hb_audio_config_t * audio;
+    audio = (hb_audio_config_t *) hb_list_audio_config_item( fTitle->list_audio, [audiotrackPopUp indexOfSelectedItem] - 1 );
+    int inputbitrate = audio->in.bitrate / 1000;
+    int inputsamplerate = audio->in.samplerate;
+    
     if ([[mixdownPopUp selectedItem] tag] != HB_ACODEC_AC3)
     {
         [bitratePopUp removeAllItems];
@@ -3486,31 +3520,32 @@ the user is using "Custom" settings by determining the sender*/
     /* populate and set the sample rate popup */
     /* Audio samplerate */
     [sampleratePopUp removeAllItems];
+    /* we create a same as source selection (Auto) so that we can choose to use the input sample rate */
+    NSMenuItem *menuItem = [[sampleratePopUp menu] addItemWithTitle: @"Auto" action: NULL keyEquivalent: @""];
+    [menuItem setTag: inputsamplerate];
+    
     for( int i = 0; i < hb_audio_rates_count; i++ )
     {
         NSMenuItem *menuItem = [[sampleratePopUp menu] addItemWithTitle:
-                                    [NSString stringWithCString: hb_audio_rates[i].string]
-                                                                  action: NULL keyEquivalent: @""];
+                                [NSString stringWithCString: hb_audio_rates[i].string]
+                                                                 action: NULL keyEquivalent: @""];
         [menuItem setTag: hb_audio_rates[i].rate];
     }
-    /* We use 48 hz as the default sample rate as almost every dvd made uses that as source
-    * and there is no compelling reason to use anything else as default
+    /* We use the input sample rate as the default sample rate as downsampling just makes audio worse
+    * and there is no compelling reason to use anything else as default, though the users default
+    * preset will likely override any setting chosen here.
     */
-    [sampleratePopUp selectItemWithTitle: @"48"];
+    [sampleratePopUp selectItemWithTag: inputsamplerate];
+    
     
-    /* Since AC3 Pass Thru uses the input ac3 bitrate and a sample rate of 48, we get the input tracks
+    /* Since AC3 Pass Thru uses the input ac3 bitrate and sample rate, we get the input tracks
     * bitrate and dispay it in the bitrate popup even though libhb happily ignores any bitrate input from
     * the gui. We do this for better user feedback in the audio tab as well as the queue for the most part
     */
     if ([[mixdownPopUp selectedItem] tag] == HB_ACODEC_AC3)
     {
-        /* make sure we have a selected title before continuing */
-        if (fTitle == NULL) return;
+        
         /* lets also set the bitrate popup to the input bitrate as thats what passthru will use */
-        /* get the audio */
-        hb_audio_config_t * audio;
-        audio = (hb_audio_config_t *) hb_list_audio_config_item( fTitle->list_audio, [audiotrackPopUp indexOfSelectedItem] - 1 );
-        int inputbitrate = audio->in.bitrate / 1000;
         [bitratePopUp removeAllItems];
         NSMenuItem *menuItem = [[bitratePopUp menu] addItemWithTitle:
                                 [NSString stringWithFormat:@"%d", inputbitrate]
@@ -3769,9 +3804,9 @@ if (item == nil)
         /* initialize the tooltip contents variable */
         NSString *loc_tip;
         /* if there is a description for the preset, we show it in the tooltip */
-        if ([item valueForKey:@"PresetDescription"])
+        if ([item objectForKey:@"PresetDescription"])
         {
-            loc_tip = [NSString stringWithFormat: @"%@",[item valueForKey:@"PresetDescription"]];
+            loc_tip = [item objectForKey:@"PresetDescription"];
             return (loc_tip);
         }
         else
@@ -3887,107 +3922,107 @@ if (item == nil)
 
 - (IBAction)selectPreset:(id)sender
 {
-    
+
     if ([fPresetsOutlineView selectedRow] >= 0)
     {
         chosenPreset = [fPresetsOutlineView itemAtRow:[fPresetsOutlineView selectedRow]];
         /* we set the preset display field in main window here */
-        [fPresetSelectedDisplay setStringValue: [NSString stringWithFormat: @"%@",[chosenPreset valueForKey:@"PresetName"]]];
+        [fPresetSelectedDisplay setStringValue:[chosenPreset objectForKey:@"PresetName"]];
         if ([[chosenPreset objectForKey:@"Default"] intValue] == 1)
         {
-            [fPresetSelectedDisplay setStringValue: [NSString stringWithFormat: @"%@ (Default)",[chosenPreset valueForKey:@"PresetName"]]];
+            [fPresetSelectedDisplay setStringValue:[NSString stringWithFormat:@"%@ (Default)", [chosenPreset objectForKey:@"PresetName"]]];
         }
         else
         {
-            [fPresetSelectedDisplay setStringValue: [NSString stringWithFormat: @"%@",[chosenPreset valueForKey:@"PresetName"]]];
+            [fPresetSelectedDisplay setStringValue:[chosenPreset objectForKey:@"PresetName"]];
         }
         /* File Format */
-        [fDstFormatPopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"FileFormat"]]];
-        [self formatPopUpChanged: NULL];
-        
+        [fDstFormatPopUp selectItemWithTitle:[chosenPreset objectForKey:@"FileFormat"]];
+        [self formatPopUpChanged:nil];
+
         /* Chapter Markers*/
         [fCreateChapterMarkers setState:[[chosenPreset objectForKey:@"ChapterMarkers"] intValue]];
         /* Allow Mpeg4 64 bit formatting +4GB file sizes */
         [fDstMp4LargeFileCheck setState:[[chosenPreset objectForKey:@"Mp4LargeFile"] intValue]];
         /* Mux mp4 with http optimization */
         [fDstMp4HttpOptFileCheck setState:[[chosenPreset objectForKey:@"Mp4HttpOptimize"] intValue]];
-        
+
         /* Video encoder */
         /* We set the advanced opt string here if applicable*/
-        [fAdvancedOptions setOptions: [NSString stringWithFormat:[chosenPreset valueForKey:@"x264Option"]]];
+        [fAdvancedOptions setOptions:[chosenPreset objectForKey:@"x264Option"]];
         /* We use a conditional to account for the new x264 encoder dropdown as well as presets made using legacy x264 settings*/
-        if ([[NSString stringWithFormat:[chosenPreset valueForKey:@"VideoEncoder"]] isEqualToString: @"x264 (h.264 Main)"] || 
-            [[NSString stringWithFormat:[chosenPreset valueForKey:@"VideoEncoder"]] isEqualToString: @"x264 (h.264 iPod)"] || 
-            [[NSString stringWithFormat:[chosenPreset valueForKey:@"VideoEncoder"]] isEqualToString: @"x264"])
+        if ([[chosenPreset objectForKey:@"VideoEncoder"] isEqualToString:@"x264 (h.264 Main)"] ||
+            [[chosenPreset objectForKey:@"VideoEncoder"] isEqualToString:@"x264 (h.264 iPod)"] ||
+            [[chosenPreset objectForKey:@"VideoEncoder"] isEqualToString:@"x264"])
         {
-            [fVidEncoderPopUp selectItemWithTitle: [NSString stringWithFormat:@"H.264 (x264)"]];
+            [fVidEncoderPopUp selectItemWithTitle:@"H.264 (x264)"];
             /* special case for legacy preset to check the new fDstMp4HttpOptFileCheck checkbox to set the ipod atom */
-            if ([[NSString stringWithFormat:[chosenPreset valueForKey:@"VideoEncoder"]] isEqualToString: @"x264 (h.264 iPod)"])
+            if ([[chosenPreset objectForKey:@"VideoEncoder"] isEqualToString:@"x264 (h.264 iPod)"])
             {
                 [fDstMp4iPodFileCheck setState:NSOnState];
                 /* We also need to add "level=30:" to the advanced opts string to set the correct level for the iPod when
                  encountering a legacy preset as it used to be handled separately from the opt string*/
-                [fAdvancedOptions setOptions: [NSString stringWithFormat:[@"level=30:" stringByAppendingString:[fAdvancedOptions optionsString]]]];
+                [fAdvancedOptions setOptions:[@"level=30:" stringByAppendingString:[fAdvancedOptions optionsString]]];
             }
             else
             {
                 [fDstMp4iPodFileCheck setState:NSOffState];
             }
         }
-        else if ([[NSString stringWithFormat:[chosenPreset valueForKey:@"VideoEncoder"]] isEqualToString: @"FFmpeg"])
+        else if ([[chosenPreset objectForKey:@"VideoEncoder"] isEqualToString:@"FFmpeg"])
         {
-            [fVidEncoderPopUp selectItemWithTitle: [NSString stringWithFormat:@"MPEG-4 (FFmpeg)"]];
+            [fVidEncoderPopUp selectItemWithTitle:@"MPEG-4 (FFmpeg)"];
         }
-        else if ([[NSString stringWithFormat:[chosenPreset valueForKey:@"VideoEncoder"]] isEqualToString: @"XviD"])
+        else if ([[chosenPreset objectForKey:@"VideoEncoder"] isEqualToString:@"XviD"])
         {
-            [fVidEncoderPopUp selectItemWithTitle: [NSString stringWithFormat:@"MPEG-4 (XviD)"]];
+            [fVidEncoderPopUp selectItemWithTitle:@"MPEG-4 (XviD)"];
         }
         else
         {
-            [fVidEncoderPopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"VideoEncoder"]]];
+            [fVidEncoderPopUp selectItemWithTitle:[chosenPreset objectForKey:@"VideoEncoder"]];
         }
-        
+
         /* Lets run through the following functions to get variables set there */
-        [self videoEncoderPopUpChanged: NULL];
+        [self videoEncoderPopUpChanged:nil];
         /* Set the state of ipod compatible with Mp4iPodCompatible. Only for x264*/
         [fDstMp4iPodFileCheck setState:[[chosenPreset objectForKey:@"Mp4iPodCompatible"] intValue]];
-        [self calculateBitrate: NULL];
-        
+        [self calculateBitrate:nil];
+
         /* Video quality */
         [fVidQualityMatrix selectCellAtRow:[[chosenPreset objectForKey:@"VideoQualityType"] intValue] column:0];
-        
-        [fVidTargetSizeField setStringValue: [NSString stringWithFormat:[chosenPreset valueForKey:@"VideoTargetSize"]]];
-        [fVidBitrateField setStringValue: [NSString stringWithFormat:[chosenPreset valueForKey:@"VideoAvgBitrate"]]];
-        [fVidQualitySlider setFloatValue: [[chosenPreset valueForKey:@"VideoQualitySlider"] floatValue]];
-        
-        [self videoMatrixChanged: NULL];
-        
+
+        [fVidTargetSizeField setStringValue:[chosenPreset objectForKey:@"VideoTargetSize"]];
+        [fVidBitrateField setStringValue:[chosenPreset objectForKey:@"VideoAvgBitrate"]];
+        [fVidQualitySlider setFloatValue:[[chosenPreset objectForKey:@"VideoQualitySlider"] floatValue]];
+
+        [self videoMatrixChanged:nil];
+
         /* Video framerate */
         /* For video preset video framerate, we want to make sure that Same as source does not conflict with the
          detected framerate in the fVidRatePopUp so we use index 0*/
-        if ([[NSString stringWithFormat:[chosenPreset valueForKey:@"VideoFramerate"]] isEqualToString: @"Same as source"])
+        if ([[chosenPreset objectForKey:@"VideoFramerate"] isEqualToString:@"Same as source"])
         {
             [fVidRatePopUp selectItemAtIndex: 0];
         }
         else
         {
-            [fVidRatePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"VideoFramerate"]]];
+            [fVidRatePopUp selectItemWithTitle:[chosenPreset objectForKey:@"VideoFramerate"]];
         }
-        
+
         /* GrayScale */
         [fVidGrayscaleCheck setState:[[chosenPreset objectForKey:@"VideoGrayScale"] intValue]];
-        
+
         /* 2 Pass Encoding */
         [fVidTwoPassCheck setState:[[chosenPreset objectForKey:@"VideoTwoPass"] intValue]];
-        [self twoPassCheckboxChanged: NULL];
+        [self twoPassCheckboxChanged:nil];
         /* Turbo 1st pass for 2 Pass Encoding */
         [fVidTurboPassCheck setState:[[chosenPreset objectForKey:@"VideoTurboTwoPass"] intValue]];
-        
+
         /*Audio*/
-        if ([chosenPreset valueForKey:@"FileCodecs"])
+        if ([chosenPreset objectForKey:@"FileCodecs"])
         {
             /* We need to handle the audio codec popup by determining what was chosen from the deprecated Codecs PopUp for past presets*/
-            if ([[NSString stringWithFormat:[chosenPreset valueForKey:@"FileCodecs"]] isEqualToString: @"AVC/H.264 Video / AAC + AC3 Audio"])
+            if ([[chosenPreset objectForKey:@"FileCodecs"] isEqualToString: @"AVC/H.264 Video / AAC + AC3 Audio"])
             {
                 /* We need to address setting languages etc. here in the new multi track audio panel */
                 /* Track One set here */
@@ -4005,8 +4040,8 @@ if (item == nil)
                 [fAudTrack2CodecPopUp selectItemWithTitle: @"AC3 Passthru"];
                 [self audioTrackPopUpChanged: fAudTrack2CodecPopUp];
             }
-            else if ([[NSString stringWithFormat:[chosenPreset valueForKey:@"FileCodecs"]] isEqualToString: @"MPEG-4 Video / AAC Audio"] ||
-                     [[NSString stringWithFormat:[chosenPreset valueForKey:@"FileCodecs"]] isEqualToString: @"AVC/H.264 Video / AAC Audio"])
+            else if ([[chosenPreset objectForKey:@"FileCodecs"] isEqualToString:@"MPEG-4 Video / AAC Audio"] ||
+                     [[chosenPreset objectForKey:@"FileCodecs"] isEqualToString:@"AVC/H.264 Video / AAC Audio"])
             {
                 if ([fAudLang1PopUp indexOfSelectedItem] > 0)
                 {
@@ -4029,8 +4064,8 @@ if (item == nil)
                     [self audioTrackPopUpChanged: fAudTrack4CodecPopUp];
                 }
             }
-            else if ([[NSString stringWithFormat:[chosenPreset valueForKey:@"FileCodecs"]] isEqualToString: @"MPEG-4 Video / AC-3 Audio"] ||
-                     [[NSString stringWithFormat:[chosenPreset valueForKey:@"FileCodecs"]] isEqualToString: @"AVC/H.264 Video / AC-3 Audio"])
+            else if ([[chosenPreset objectForKey:@"FileCodecs"] isEqualToString:@"MPEG-4 Video / AC-3 Audio"] ||
+                     [[chosenPreset objectForKey:@"FileCodecs"] isEqualToString:@"AVC/H.264 Video / AC-3 Audio"])
             {
                 if ([fAudLang1PopUp indexOfSelectedItem] > 0)
                 {
@@ -4053,8 +4088,8 @@ if (item == nil)
                     [self audioTrackPopUpChanged: fAudTrack4CodecPopUp];
                 }
             }
-            else if ([[NSString stringWithFormat:[chosenPreset valueForKey:@"FileCodecs"]] isEqualToString: @"MPEG-4 Video / MP3 Audio"] ||
-                     [[NSString stringWithFormat:[chosenPreset valueForKey:@"FileCodecs"]] isEqualToString: @"AVC/H.264 Video / MP3 Audio"])
+            else if ([[chosenPreset objectForKey:@"FileCodecs"] isEqualToString:@"MPEG-4 Video / MP3 Audio"] ||
+                     [[chosenPreset objectForKey:@"FileCodecs"] isEqualToString:@"AVC/H.264 Video / MP3 Audio"])
             {
                 if ([fAudLang1PopUp indexOfSelectedItem] > 0)
                 {
@@ -4077,7 +4112,7 @@ if (item == nil)
                     [self audioTrackPopUpChanged: fAudTrack4CodecPopUp];
                 }
             }
-            else if ([[NSString stringWithFormat:[chosenPreset valueForKey:@"FileCodecs"]] isEqualToString: @"MPEG-4 Video / Vorbis Audio"])
+            else if ([[chosenPreset objectForKey:@"FileCodecs"] isEqualToString:@"MPEG-4 Video / Vorbis Audio"])
             {
                 if ([fAudLang1PopUp indexOfSelectedItem] > 0)
                 {
@@ -4100,127 +4135,196 @@ if (item == nil)
                     [self audioTrackPopUpChanged: fAudTrack4CodecPopUp];
                 }
             }
-            /* We detect here if we have the old audio sample rate and if so we apply samplerate and bitrate to the existing four tracks if chosen 
+            /* We detect here if we have the old audio sample rate and if so we apply samplerate and bitrate to the existing four tracks if chosen
             * UNLESS the CodecPopUp is AC3 in which case the preset values are ignored in favor of rates set in audioTrackMixdownChanged*/
-            if ([chosenPreset valueForKey:@"AudioSampleRate"])
+            if ([chosenPreset objectForKey:@"AudioSampleRate"])
             {
                 if ([fAudLang1PopUp indexOfSelectedItem] > 0 && [fAudTrack1CodecPopUp titleOfSelectedItem] != @"AC3 Passthru")
                 {
-                    [fAudTrack1RatePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"AudioSampleRate"]]];
-                    [fAudTrack1BitratePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"AudioBitRate"]]];
+                    [fAudTrack1RatePopUp selectItemWithTitle:[chosenPreset objectForKey:@"AudioSampleRate"]];
+                    [fAudTrack1BitratePopUp selectItemWithTitle:[chosenPreset objectForKey:@"AudioBitRate"]];
                 }
                 if ([fAudLang2PopUp indexOfSelectedItem] > 0 && [fAudTrack2CodecPopUp titleOfSelectedItem] != @"AC3 Passthru")
                 {
-                    [fAudTrack2RatePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"AudioSampleRate"]]];
-                    [fAudTrack2BitratePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"AudioBitRate"]]];
+                    [fAudTrack2RatePopUp selectItemWithTitle:[chosenPreset objectForKey:@"AudioSampleRate"]];
+                    [fAudTrack2BitratePopUp selectItemWithTitle:[chosenPreset objectForKey:@"AudioBitRate"]];
                 }
                 if ([fAudLang3PopUp indexOfSelectedItem] > 0 && [fAudTrack3CodecPopUp titleOfSelectedItem] != @"AC3 Passthru")
                 {
-                    [fAudTrack3RatePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"AudioSampleRate"]]];
-                    [fAudTrack3BitratePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"AudioBitRate"]]];
+                    [fAudTrack3RatePopUp selectItemWithTitle:[chosenPreset objectForKey:@"AudioSampleRate"]];
+                    [fAudTrack3BitratePopUp selectItemWithTitle:[chosenPreset objectForKey:@"AudioBitRate"]];
                 }
                 if ([fAudLang4PopUp indexOfSelectedItem] > 0 && [fAudTrack4CodecPopUp titleOfSelectedItem] != @"AC3 Passthru")
                 {
-                    [fAudTrack4RatePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"AudioSampleRate"]]];
-                    [fAudTrack4BitratePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"AudioBitRate"]]];
+                    [fAudTrack4RatePopUp selectItemWithTitle:[chosenPreset objectForKey:@"AudioSampleRate"]];
+                    [fAudTrack4BitratePopUp selectItemWithTitle:[chosenPreset objectForKey:@"AudioBitRate"]];
                 }
             }
             /* We detect here if we have the old DRC Slider and if so we apply it to the existing four tracks if chosen */
-            if ([chosenPreset valueForKey:@"AudioDRCSlider"])
+            if ([chosenPreset objectForKey:@"AudioDRCSlider"])
             {
                 if ([fAudLang1PopUp indexOfSelectedItem] > 0)
                 {
-                    [fAudTrack1DrcSlider setFloatValue: [[chosenPreset valueForKey:@"AudioDRCSlider"] floatValue]];
+                    [fAudTrack1DrcSlider setFloatValue:[[chosenPreset objectForKey:@"AudioDRCSlider"] floatValue]];
                     [self audioDRCSliderChanged: fAudTrack1DrcSlider];
                 }
                 if ([fAudLang2PopUp indexOfSelectedItem] > 0)
                 {
-                    [fAudTrack2DrcSlider setFloatValue: [[chosenPreset valueForKey:@"AudioDRCSlider"] floatValue]];
+                    [fAudTrack2DrcSlider setFloatValue:[[chosenPreset objectForKey:@"AudioDRCSlider"] floatValue]];
                     [self audioDRCSliderChanged: fAudTrack2DrcSlider];
                 }
                 if ([fAudLang3PopUp indexOfSelectedItem] > 0)
                 {
-                    [fAudTrack3DrcSlider setFloatValue: [[chosenPreset valueForKey:@"AudioDRCSlider"] floatValue]];
+                    [fAudTrack3DrcSlider setFloatValue:[[chosenPreset objectForKey:@"AudioDRCSlider"] floatValue]];
                     [self audioDRCSliderChanged: fAudTrack3DrcSlider];
                 }
                 if ([fAudLang4PopUp indexOfSelectedItem] > 0)
                 {
-                    [fAudTrack4DrcSlider setFloatValue: [[chosenPreset valueForKey:@"AudioDRCSlider"] floatValue]];
+                    [fAudTrack4DrcSlider setFloatValue:[[chosenPreset objectForKey:@"AudioDRCSlider"] floatValue]];
                     [self audioDRCSliderChanged: fAudTrack4DrcSlider];
                 }
             }
         }
         else // since there was no codecs key in the preset we know we can use new multi-audio track presets
         {
-            if ([chosenPreset valueForKey:@"Audio1Track"] > 0)
+            if ([chosenPreset objectForKey:@"Audio1Track"] > 0)
             {
                 if ([fAudLang1PopUp indexOfSelectedItem] == 0)
                 {
                     [fAudLang1PopUp selectItemAtIndex: 1];
-                    [self audioTrackPopUpChanged: fAudLang1PopUp];
                 }
-                [fAudTrack1CodecPopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Audio1Encoder"]]];
+                [self audioTrackPopUpChanged: fAudLang1PopUp];
+                [fAudTrack1CodecPopUp selectItemWithTitle:[chosenPreset objectForKey:@"Audio1Encoder"]];
                 [self audioTrackPopUpChanged: fAudTrack1CodecPopUp];
-                [fAudTrack1MixPopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Audio1Mixdown"]]];
-                [fAudTrack1RatePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Audio1Samplerate"]]];
-                [fAudTrack1BitratePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Audio1Bitrate"]]];
-                [fAudTrack1DrcSlider setFloatValue: [[chosenPreset valueForKey:@"Audio1TrackDRCSlider"] floatValue]];
+                [fAudTrack1MixPopUp selectItemWithTitle:[chosenPreset objectForKey:@"Audio1Mixdown"]];
+                /* check to see if the selections was available, if not, rerun audioTrackPopUpChanged using the codec to just set the default
+                 * mixdown*/
+                if  ([fAudTrack1MixPopUp selectedItem] == nil)
+                {
+                    [self audioTrackPopUpChanged: fAudTrack1CodecPopUp];
+                }
+                [fAudTrack1RatePopUp selectItemWithTitle:[chosenPreset objectForKey:@"Audio1Samplerate"]];
+                /* We set the presets bitrate if it is *not* an AC3 track since that uses the input bitrate */
+                if (![[chosenPreset objectForKey:@"Audio1Encoder"] isEqualToString:@"AC3 Passthru"])
+                {
+                    [fAudTrack1BitratePopUp selectItemWithTitle:[chosenPreset objectForKey:@"Audio1Bitrate"]];
+                }
+                [fAudTrack1DrcSlider setFloatValue:[[chosenPreset objectForKey:@"Audio1TrackDRCSlider"] floatValue]];
                 [self audioDRCSliderChanged: fAudTrack1DrcSlider];
             }
-            if ([chosenPreset valueForKey:@"Audio2Track"] > 0)
+            if ([chosenPreset objectForKey:@"Audio2Track"] > 0)
             {
                 if ([fAudLang2PopUp indexOfSelectedItem] == 0)
                 {
                     [fAudLang2PopUp selectItemAtIndex: 1];
-                    [self audioTrackPopUpChanged: fAudLang2PopUp];
                 }
-                [fAudTrack2CodecPopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Audio2Encoder"]]];
+                [self audioTrackPopUpChanged: fAudLang2PopUp];
+                [fAudTrack2CodecPopUp selectItemWithTitle:[chosenPreset objectForKey:@"Audio2Encoder"]];
                 [self audioTrackPopUpChanged: fAudTrack2CodecPopUp];
-                [fAudTrack2MixPopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Audio2Mixdown"]]];
-                [fAudTrack2RatePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Audio2Samplerate"]]];
-                [fAudTrack2BitratePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Audio2Bitrate"]]];
-                [fAudTrack2DrcSlider setFloatValue: [[chosenPreset valueForKey:@"Audio2TrackDRCSlider"] floatValue]];
+                [fAudTrack2MixPopUp selectItemWithTitle:[chosenPreset objectForKey:@"Audio2Mixdown"]];
+                /* check to see if the selections was available, if not, rerun audioTrackPopUpChanged using the codec to just set the default
+                 * mixdown*/
+                if  ([fAudTrack2MixPopUp selectedItem] == nil)
+                {
+                    [self audioTrackPopUpChanged: fAudTrack2CodecPopUp];
+                }
+                [fAudTrack2RatePopUp selectItemWithTitle:[chosenPreset objectForKey:@"Audio2Samplerate"]];
+                /* We set the presets bitrate if it is *not* an AC3 track since that uses the input bitrate */
+                if (![[chosenPreset objectForKey:@"Audio2Encoder"] isEqualToString:@"AC3 Passthru"])
+                {
+                    [fAudTrack2BitratePopUp selectItemWithTitle:[chosenPreset objectForKey:@"Audio2Bitrate"]];
+                }
+                [fAudTrack2DrcSlider setFloatValue:[[chosenPreset objectForKey:@"Audio2TrackDRCSlider"] floatValue]];
                 [self audioDRCSliderChanged: fAudTrack2DrcSlider];
             }
-            if ([chosenPreset valueForKey:@"Audio3Track"] > 0)
+            if ([chosenPreset objectForKey:@"Audio3Track"] > 0)
             {
                 if ([fAudLang3PopUp indexOfSelectedItem] == 0)
                 {
                     [fAudLang3PopUp selectItemAtIndex: 1];
-                    [self audioTrackPopUpChanged: fAudLang3PopUp];
                 }
-                [fAudTrack3CodecPopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Audio3Encoder"]]];
+                [self audioTrackPopUpChanged: fAudLang3PopUp];
+                [fAudTrack3CodecPopUp selectItemWithTitle:[chosenPreset objectForKey:@"Audio3Encoder"]];
                 [self audioTrackPopUpChanged: fAudTrack3CodecPopUp];
-                [fAudTrack3MixPopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Audio3Mixdown"]]];
-                [fAudTrack3RatePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Audio3Samplerate"]]];
-                [fAudTrack3BitratePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Audio3Bitrate"]]];
-                [fAudTrack3DrcSlider setFloatValue: [[chosenPreset valueForKey:@"Audio3TrackDRCSlider"] floatValue]];
+                [fAudTrack3MixPopUp selectItemWithTitle:[chosenPreset objectForKey:@"Audio3Mixdown"]];
+                /* check to see if the selections was available, if not, rerun audioTrackPopUpChanged using the codec to just set the default
+                 * mixdown*/
+                if  ([fAudTrack3MixPopUp selectedItem] == nil)
+                {
+                    [self audioTrackPopUpChanged: fAudTrack3CodecPopUp];
+                }
+                [fAudTrack3RatePopUp selectItemWithTitle:[chosenPreset objectForKey:@"Audio3Samplerate"]];
+                /* We set the presets bitrate if it is *not* an AC3 track since that uses the input bitrate */
+                if (![[chosenPreset objectForKey:@"Audio3Encoder"] isEqualToString: @"AC3 Passthru"])
+                {
+                    [fAudTrack3BitratePopUp selectItemWithTitle:[chosenPreset objectForKey:@"Audio3Bitrate"]];
+                }
+                [fAudTrack3DrcSlider setFloatValue:[[chosenPreset objectForKey:@"Audio3TrackDRCSlider"] floatValue]];
                 [self audioDRCSliderChanged: fAudTrack3DrcSlider];
             }
-            if ([chosenPreset valueForKey:@"Audio4Track"] > 0)
+            if ([chosenPreset objectForKey:@"Audio4Track"] > 0)
             {
                 if ([fAudLang4PopUp indexOfSelectedItem] == 0)
                 {
                     [fAudLang4PopUp selectItemAtIndex: 1];
-                    [self audioTrackPopUpChanged: fAudLang4PopUp];
                 }
-                [fAudTrack4CodecPopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Audio4Encoder"]]];
+                [self audioTrackPopUpChanged: fAudLang4PopUp];
+                [fAudTrack4CodecPopUp selectItemWithTitle:[chosenPreset objectForKey:@"Audio4Encoder"]];
                 [self audioTrackPopUpChanged: fAudTrack4CodecPopUp];
-                [fAudTrack4MixPopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Audio4Mixdown"]]];
-                [fAudTrack4RatePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Audio4Samplerate"]]];
-                [fAudTrack4BitratePopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Audio4Bitrate"]]];
-                [fAudTrack4DrcSlider setFloatValue: [[chosenPreset valueForKey:@"Audio4TrackDRCSlider"] floatValue]];
+                [fAudTrack4MixPopUp selectItemWithTitle:[chosenPreset objectForKey:@"Audio4Mixdown"]];
+                /* check to see if the selections was available, if not, rerun audioTrackPopUpChanged using the codec to just set the default
+                 * mixdown*/
+                if  ([fAudTrack4MixPopUp selectedItem] == nil)
+                {
+                    [self audioTrackPopUpChanged: fAudTrack4CodecPopUp];
+                }
+                [fAudTrack4RatePopUp selectItemWithTitle:[chosenPreset objectForKey:@"Audio4Samplerate"]];
+                /* We set the presets bitrate if it is *not* an AC3 track since that uses the input bitrate */
+                if (![[chosenPreset objectForKey:@"Audio4Encoder"] isEqualToString:@"AC3 Passthru"])
+                {
+                    [fAudTrack4BitratePopUp selectItemWithTitle:[chosenPreset objectForKey:@"Audio4Bitrate"]];
+                }
+                [fAudTrack4DrcSlider setFloatValue:[[chosenPreset objectForKey:@"Audio4TrackDRCSlider"] floatValue]];
                 [self audioDRCSliderChanged: fAudTrack4DrcSlider];
             }
-            
-            
+
+
         }
-        
+
+        /* We now cleanup any extra audio tracks that may be previously set if we need to, we do it here so we don't have to
+         * duplicate any code for legacy presets.*/
+        /* First we handle the legacy Codecs crazy AVC/H.264 Video / AAC + AC3 Audio atv hybrid */
+        if ([chosenPreset objectForKey:@"FileCodecs"] && [[chosenPreset objectForKey:@"FileCodecs"] isEqualToString:@"AVC/H.264 Video / AAC + AC3 Audio"])
+        {
+            [fAudLang3PopUp selectItemAtIndex: 0];
+            [self audioTrackPopUpChanged: fAudLang3PopUp];
+            [fAudLang4PopUp selectItemAtIndex: 0];
+            [self audioTrackPopUpChanged: fAudLang4PopUp];
+        }
+        else
+        {
+            if (![chosenPreset objectForKey:@"Audio2Track"] || [chosenPreset objectForKey:@"Audio2Track"] == 0)
+            {
+                [fAudLang2PopUp selectItemAtIndex: 0];
+                [self audioTrackPopUpChanged: fAudLang2PopUp];
+            }
+            if (![chosenPreset objectForKey:@"Audio3Track"] || [chosenPreset objectForKey:@"Audio3Track"] > 0)
+            {
+                [fAudLang3PopUp selectItemAtIndex: 0];
+                [self audioTrackPopUpChanged: fAudLang3PopUp];
+            }
+            if (![chosenPreset objectForKey:@"Audio4Track"] || [chosenPreset objectForKey:@"Audio4Track"] > 0)
+            {
+                [fAudLang4PopUp selectItemAtIndex: 0];
+                [self audioTrackPopUpChanged: fAudLang4PopUp];
+            }
+        }
+
         /*Subtitles*/
-        [fSubPopUp selectItemWithTitle: [NSString stringWithFormat:[chosenPreset valueForKey:@"Subtitles"]]];
+        [fSubPopUp selectItemWithTitle:[chosenPreset objectForKey:@"Subtitles"]];
         /* Forced Subtitles */
         [fSubForcedCheck setState:[[chosenPreset objectForKey:@"SubtitlesForced"] intValue]];
-           
+
         /* Picture Settings */
         /* Note: objectForKey:@"UsesPictureSettings" now refers to picture size, this encompasses:
          * height, width, keep ar, anamorphic and crop settings.
@@ -4238,7 +4342,7 @@ if (item == nil)
             if ([[chosenPreset objectForKey:@"UsesPictureSettings"]  intValue] == 2 || [[chosenPreset objectForKey:@"UsesMaxPictureSettings"]  intValue] == 1)
             {
                 /* Use Max Picture settings for whatever the dvd is.*/
-                [self revertPictureSizeToMax: NULL];
+                [self revertPictureSizeToMax:nil];
                 job->keep_ratio = [[chosenPreset objectForKey:@"PictureKeepRatio"]  intValue];
                 if (job->keep_ratio == 1)
                 {
@@ -4364,13 +4468,14 @@ if (item == nil)
                     else
                     {
                         [fPictureController setDeblock:0];
-                    }             
-                    [self calculatePictureSizing: NULL];
+                    }
+
+                    [self calculatePictureSizing:nil];
                 }
-                
+
             }
-            
-            
+
+
         }
         /* If the preset has an objectForKey:@"UsesPictureFilters", then we know it is a newer style filters preset
          * and handle the filters here depending on whether or not the preset specifies applying the filter.
@@ -4431,10 +4536,20 @@ if (item == nil)
             else
             {
                 [fPictureController setDeblock:0];
-            }             
+            }
+            /* Decomb */
+            /* Even though we currently allow for a custom setting for decomb, ultimately it will only have Off and
+             * Default so we just pay attention to anything greater than 0 as 1 (Default). 0 is Off. */
+            if ([[chosenPreset objectForKey:@"PictureDecomb"] intValue] > 0)
+            {
+                [fPictureController setDecomb:1];
+            }
+            else
+            {
+                [fPictureController setDecomb:0];
+            }
         }
-        [self calculatePictureSizing: NULL];
-        [[fPresetsActionMenu itemAtIndex:0] setEnabled: YES];
+        [self calculatePictureSizing:nil];
     }
 }
 
@@ -4449,16 +4564,16 @@ if (item == nil)
     UserPresetsFile = @"~/Library/Application Support/HandBrake/UserPresets.plist";
        UserPresetsFile = [[UserPresetsFile stringByExpandingTildeInPath]retain];
     /* We check for the presets.plist */
-       if ([fileManager fileExistsAtPath:UserPresetsFile] == 0) 
+       if ([fileManager fileExistsAtPath:UserPresetsFile] == 0)
        {
                [fileManager createFileAtPath:UserPresetsFile contents:nil attributes:nil];
        }
-               
+
        UserPresets = [[NSMutableArray alloc] initWithContentsOfFile:UserPresetsFile];
-       if (nil == UserPresets) 
+       if (nil == UserPresets)
        {
                UserPresets = [[NSMutableArray alloc] init];
-               [self addFactoryPresets:NULL];
+               [self addFactoryPresets:nil];
        }
        [fPresetsOutlineView reloadData];
 }
@@ -4481,7 +4596,7 @@ if (item == nil)
        [fPresetNewName setStringValue: @""];
        [fPresetNewDesc setStringValue: @""];
        /* Show the panel */
-       [NSApp beginSheet: fAddPresetPanel modalForWindow: fWindow modalDelegate: NULL didEndSelector: NULL contextInfo: NULL];
+       [NSApp beginSheet:fAddPresetPanel modalForWindow:fWindow modalDelegate:nil didEndSelector:NULL contextInfo:NULL];
 }
 
 - (IBAction) closeAddPresetPanel: (id) sender
@@ -4499,8 +4614,8 @@ if (item == nil)
         /* Here we create a custom user preset */
         [UserPresets addObject:[self createPreset]];
         [self addPreset];
-        
-        [self closeAddPresetPanel:NULL];
+
+        [self closeAddPresetPanel:nil];
     }
 }
 - (void)addPreset
@@ -4569,22 +4684,22 @@ if (item == nil)
     [preset setObject:[NSNumber numberWithInt:[fDstMp4HttpOptFileCheck state]] forKey:@"Mp4HttpOptimize"];
     /* Add iPod uuid atom */
     [preset setObject:[NSNumber numberWithInt:[fDstMp4iPodFileCheck state]] forKey:@"Mp4iPodCompatible"];
-    
+
     /* Codecs */
        /* Video encoder */
        [preset setObject:[fVidEncoderPopUp titleOfSelectedItem] forKey:@"VideoEncoder"];
        /* x264 Option String */
        [preset setObject:[fAdvancedOptions optionsString] forKey:@"x264Option"];
-       
+
        [preset setObject:[NSNumber numberWithInt:[fVidQualityMatrix selectedRow]] forKey:@"VideoQualityType"];
        [preset setObject:[fVidTargetSizeField stringValue] forKey:@"VideoTargetSize"];
        [preset setObject:[fVidBitrateField stringValue] forKey:@"VideoAvgBitrate"];
        [preset setObject:[NSNumber numberWithFloat:[fVidQualitySlider floatValue]] forKey:@"VideoQualitySlider"];
-       
+
        /* Video framerate */
     if ([fVidRatePopUp indexOfSelectedItem] == 0) // Same as source is selected
        {
-    [preset setObject:[NSString stringWithFormat: @"Same as source"] forKey:@"VideoFramerate"];
+        [preset setObject:@"Same as source" forKey:@"VideoFramerate"];
     }
     else // we can record the actual titleOfSelectedItem
     {
@@ -4618,7 +4733,9 @@ if (item == nil)
        [preset setObject:[NSNumber numberWithInt:[fPictureController detelecine]] forKey:@"PictureDetelecine"];
     [preset setObject:[NSNumber numberWithInt:[fPictureController vfr]] forKey:@"VFR"];
        [preset setObject:[NSNumber numberWithInt:[fPictureController denoise]] forKey:@"PictureDenoise"];
-    [preset setObject:[NSNumber numberWithInt:[fPictureController deblock]] forKey:@"PictureDeblock"];
+    [preset setObject:[NSNumber numberWithInt:[fPictureController deblock]] forKey:@"PictureDeblock"]; 
+    [preset setObject:[NSNumber numberWithInt:[fPictureController decomb]] forKey:@"PictureDecomb"];
+    
     
     /*Audio*/
     if ([fAudLang1PopUp indexOfSelectedItem] > 0)
@@ -4676,7 +4793,7 @@ if (item == nil)
 {
     [UserPresets writeToFile:UserPresetsFile atomically:YES];
        /* We get the default preset in case it changed */
-       [self getDefaultPresets: NULL];
+       [self getDefaultPresets:nil];
 
 }
 
@@ -4775,12 +4892,12 @@ if (item == nil)
        if (presetUserDefault)
        {
        [fPresetsOutlineView selectRowIndexes:[NSIndexSet indexSetWithIndex:presetUserDefault] byExtendingSelection:NO];
-       [self selectPreset:NULL];
+       [self selectPreset:nil];
        }
        else if (presetHbDefault) //else we use the built in default presetHbDefault
        {
        [fPresetsOutlineView selectRowIndexes:[NSIndexSet indexSetWithIndex:presetHbDefault] byExtendingSelection:NO];
-       [self selectPreset:NULL];
+       [self selectPreset:nil];
        }
 }