MacGui: Fixes a small bug -- if a default value was selected twice for one of the simple integer widgets like ref frames, it'd write ref=-1 to the text field instead of writing nothing. Hopefully didn't break anything whilst fixing this.

git-svn-id: svn://svn.handbrake.fr/HandBrake/trunk@1050 b64f7644-9d1e-0410-96f1-a4d463321fa5

1 files changed, 6 insertions, 3 deletions

diff --git a/macosx/HBAdvancedController.m b/macosx/HBAdvancedController.m
index dc92f957b..930035574 100644
--- a/macosx/HBAdvancedController.m
+++ b/macosx/HBAdvancedController.m
@@ -774,7 +774,8 @@
                         }
                         else // we have a valid value to change, so change it
                         {
-                            thisOpt = [NSString stringWithFormat:@"%@=%d",optName,[sender indexOfSelectedItem]-1];
+                            if ( [sender indexOfSelectedItem] != 0 )
+                                thisOpt = [NSString stringWithFormat:@"%@=%d",optName,[sender indexOfSelectedItem]-1];
                         }
                     }
                 }
@@ -913,7 +914,8 @@
                 }            
                 else
                 {
-                    [fDisplayX264Options setStringValue:[NSString stringWithFormat:@"%@=%@", 
+                    if ( [sender indexOfSelectedItem] != 0 )
+                        [fDisplayX264Options setStringValue:[NSString stringWithFormat:@"%@=%@", 
                         [NSString stringWithFormat:optNameToChange],[NSString stringWithFormat:@"%d",[sender indexOfSelectedItem]-1]]];
                 }
             }
@@ -1039,7 +1041,8 @@
                 }
                 else
                 {
-                    [fDisplayX264Options setStringValue:[NSString stringWithFormat:@"%@:%@=%@",[NSString stringWithFormat:[fDisplayX264Options stringValue]], 
+                    if ( [sender indexOfSelectedItem] != 0 )
+                        [fDisplayX264Options setStringValue:[NSString stringWithFormat:@"%@:%@=%@",[NSString stringWithFormat:[fDisplayX264Options stringValue]], 
                         [NSString stringWithFormat:optNameToChange],[NSString stringWithFormat:@"%d",[sender indexOfSelectedItem]-1]]];
                 }
             }