From 702f4d19379c1924054b408cddb608b388c18929 Mon Sep 17 00:00:00 2001 From: Paul Chote Date: Sat, 10 Aug 2019 14:32:53 +0100 Subject: [PATCH] Remove unused return values from Show*Dropdown methods. --- .../Widgets/Logic/SettingsLogic.cs | 21 +++++++------------ 1 file changed, 7 insertions(+), 14 deletions(-) diff --git a/OpenRA.Mods.Common/Widgets/Logic/SettingsLogic.cs b/OpenRA.Mods.Common/Widgets/Logic/SettingsLogic.cs index 13f495242c..bc914ffce6 100644 --- a/OpenRA.Mods.Common/Widgets/Logic/SettingsLogic.cs +++ b/OpenRA.Mods.Common/Widgets/Logic/SettingsLogic.cs @@ -589,7 +589,7 @@ namespace OpenRA.Mods.Common.Widgets.Logic }; } - static bool ShowMouseScrollDropdown(DropDownButtonWidget dropdown, GameSettings s, bool rightMouse) + static void ShowMouseScrollDropdown(DropDownButtonWidget dropdown, GameSettings s, bool rightMouse) { var options = new Dictionary() { @@ -609,10 +609,9 @@ namespace OpenRA.Mods.Common.Widgets.Logic }; dropdown.ShowDropDown("LABEL_DROPDOWN_TEMPLATE", 500, options.Keys, setupItem); - return true; } - static bool ShowZoomModifierDropdown(DropDownButtonWidget dropdown, GameSettings s) + static void ShowZoomModifierDropdown(DropDownButtonWidget dropdown, GameSettings s) { var options = new Dictionary() { @@ -633,10 +632,9 @@ namespace OpenRA.Mods.Common.Widgets.Logic }; dropdown.ShowDropDown("LABEL_DROPDOWN_TEMPLATE", 500, options.Keys, setupItem); - return true; } - bool ShowAudioDeviceDropdown(DropDownButtonWidget dropdown, SoundDevice[] devices) + void ShowAudioDeviceDropdown(DropDownButtonWidget dropdown, SoundDevice[] devices) { var i = 0; var options = devices.ToDictionary(d => (i++).ToString(), d => d); @@ -655,10 +653,9 @@ namespace OpenRA.Mods.Common.Widgets.Logic }; dropdown.ShowDropDown("LABEL_DROPDOWN_TEMPLATE", 500, options.Keys, setupItem); - return true; } - static bool ShowWindowModeDropdown(DropDownButtonWidget dropdown, GraphicSettings s) + static void ShowWindowModeDropdown(DropDownButtonWidget dropdown, GraphicSettings s) { var options = new Dictionary() { @@ -678,10 +675,9 @@ namespace OpenRA.Mods.Common.Widgets.Logic }; dropdown.ShowDropDown("LABEL_DROPDOWN_TEMPLATE", 500, options.Keys, setupItem); - return true; } - static bool ShowLanguageDropdown(DropDownButtonWidget dropdown, IEnumerable languages) + static void ShowLanguageDropdown(DropDownButtonWidget dropdown, IEnumerable languages) { Func setupItem = (o, itemTemplate) => { @@ -694,10 +690,9 @@ namespace OpenRA.Mods.Common.Widgets.Logic }; dropdown.ShowDropDown("LABEL_DROPDOWN_TEMPLATE", 500, languages, setupItem); - return true; } - static bool ShowStatusBarsDropdown(DropDownButtonWidget dropdown, GameSettings s) + static void ShowStatusBarsDropdown(DropDownButtonWidget dropdown, GameSettings s) { var options = new Dictionary() { @@ -717,10 +712,9 @@ namespace OpenRA.Mods.Common.Widgets.Logic }; dropdown.ShowDropDown("LABEL_DROPDOWN_TEMPLATE", 500, options.Keys, setupItem); - return true; } - static bool ShowTargetLinesDropdown(DropDownButtonWidget dropdown, GameSettings s) + static void ShowTargetLinesDropdown(DropDownButtonWidget dropdown, GameSettings s) { var options = new Dictionary() { @@ -740,7 +734,6 @@ namespace OpenRA.Mods.Common.Widgets.Logic }; dropdown.ShowDropDown("LABEL_DROPDOWN_TEMPLATE", 500, options.Keys, setupItem); - return true; } void MakeMouseFocusSettingsLive()