From bb3602c3ffb501765451ac62694bb7dc45b6bb20 Mon Sep 17 00:00:00 2001 From: John Lewin Date: Tue, 5 Sep 2017 09:08:09 -0700 Subject: [PATCH] Remove redundant parameters in favor of existing instance members - Ensure local printerConnection reference is stored --- SlicerConfiguration/SliceSettingsWidget.cs | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/SlicerConfiguration/SliceSettingsWidget.cs b/SlicerConfiguration/SliceSettingsWidget.cs index a16ce6a6b..2fa1aea65 100644 --- a/SlicerConfiguration/SliceSettingsWidget.cs +++ b/SlicerConfiguration/SliceSettingsWidget.cs @@ -403,9 +403,7 @@ namespace MatterHackers.MatterControl.SlicerConfiguration addedSettingToSubGroup = true; topToBottomSettings.AddChild( CreateSettingInfoUIControls( - printerConnection, settingData, - settingsContext, copyIndex, ref tabIndexForItem)); @@ -565,9 +563,7 @@ namespace MatterHackers.MatterControl.SlicerConfiguration if (EngineMappingsMatterSlice.Instance.MapContains(settingData.SlicerConfigName)) { GuiWidget controlsForThisSetting = CreateSettingInfoUIControls( - printerConnection, settingData, - settingsContext, 0, ref tabIndexForItem); @@ -835,20 +831,8 @@ namespace MatterHackers.MatterControl.SlicerConfiguration return dataArea; } - public GuiWidget CreateSettingControl(PrinterConnection printerConnection, string sliceSettingsKey, bool isPrimarySettingsView, ref int tabIndex) - { - return CreateSettingInfoUIControls( - printerConnection, - SliceSettingsOrganizer.Instance.GetSettingsData(sliceSettingsKey), - new SettingsContext(null, NamedSettingsLayers.All), - 0, - ref tabIndex); - } - private GuiWidget CreateSettingInfoUIControls( - PrinterConnection printerConnection, SliceSettingData settingData, - SettingsContext settingsContext, int extruderIndex, ref int tabIndexForItem) {