Skip to content

Commit

Permalink
Pref Mixer: avoid redundant calls to widget update slots
Browse files Browse the repository at this point in the history
  • Loading branch information
ronso0 committed Jun 28, 2023
1 parent 94c1fb8 commit 5ccbfc0
Showing 1 changed file with 4 additions and 8 deletions.
12 changes: 4 additions & 8 deletions src/preferences/dialog/dlgprefmixer.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -77,10 +77,6 @@ DlgPrefMixer::DlgPrefMixer(
m_initializing(true) {
setupUi(this);

setUpMainEQ();

slotUpdate();

// Update the crossfader curve graph and other settings when the
// crossfader mode is changed or the slider is moved.
connect(SliderXFader,
Expand Down Expand Up @@ -161,6 +157,10 @@ DlgPrefMixer::DlgPrefMixer(
this,
&DlgPrefMixer::slotPopulateQuickEffectSelectors);

setUpMainEQ();

slotUpdate();

m_initializing = false;

// This applies the Main EQ and saves default values of previously missing
Expand Down Expand Up @@ -417,8 +417,6 @@ void DlgPrefMixer::setDefaultShelves() {
getSliderPosition(250,
SliderLoEQ->minimum(),
SliderLoEQ->maximum()));
slotLoEqSliderChanged();
slotHiEqSliderChanged();
}

void DlgPrefMixer::slotResetToDefaults() {
Expand Down Expand Up @@ -809,8 +807,6 @@ void DlgPrefMixer::slotUpdate() {
getSliderPosition(lowEqFreq,
SliderLoEQ->minimum(),
SliderLoEQ->maximum()));
slotLoEqSliderChanged();
slotHiEqSliderChanged();
}

CheckBoxInstantMainEQ->setChecked(m_instantMainEq);
Expand Down

0 comments on commit 5ccbfc0

Please sign in to comment.