Simpler #634, always use the first choice when out of bounds

cjcliffe · Mar 24, 2018 · 120d394 · 120d394
1 parent 4cd8735
commit 120d394
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/src/forms/SDRDevices/SDRDevices.cpp b/src/forms/SDRDevices/SDRDevices.cpp
@@ -502,16 +502,12 @@ std::string SDRDevicesDialog::getSelectedChoiceOption(wxPGProperty* prop, const
         if (choiceIndex >= 0 && choiceIndex < arg.options.size()) {
             //normal selection
             optionName = arg.options[choiceIndex];
-        } else if (choiceIndex >= arg.options.size()) {
-            //choose the last one of the list:
-            optionName = arg.options[arg.options.size() - 1];
-            prop->SetChoiceSelection(arg.options.size() - 1);
-        } else if (choiceIndex < 0) {
+        } else {
             //choose the first one of the list:
             optionName = arg.options[0];
             prop->SetChoiceSelection(0);
         }
-    }
+    } 
 
     return optionName;
 }