Skip to content

V
vlc-gpu
Commit 86e63937 authored by Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf
Browse files
Options

Qt4 - SPrefs: use a switch to remove stupid iterative ifs.

parent d2d0350a
Show whitespace changes
Inline Side-by-side
Showing with 10 additions and 4 deletions
modules/gui/qt4/components/simple_preferences.cpp
View file @ 86e63937
...@@ -460,6 +460,7 @@ void SPrefsPanel::updateAudioOptions( int number) ...@@ -460,6 +460,7 @@ void SPrefsPanel::updateAudioOptions( int number)
optionWidgets[fileW]->setVisible( ( value == "aout_file" ) ); optionWidgets[fileW]->setVisible( ( value == "aout_file" ) );
} }
/* Function called from the main Preferences dialog on each SPrefs Panel */
void SPrefsPanel::apply() void SPrefsPanel::apply()
{ {
msg_Dbg( p_intf, "Trying to save the %i simple panel", number ); msg_Dbg( p_intf, "Trying to save the %i simple panel", number );
...@@ -472,8 +473,9 @@ void SPrefsPanel::apply() ...@@ -472,8 +473,9 @@ void SPrefsPanel::apply()
c->doApply( p_intf ); c->doApply( p_intf );
} }
/* Devices */ switch( number )
if( number == SPrefsInputAndCodecs ) {
case SPrefsInputAndCodecs:
{ {
/* Device default selection */ /* Device default selection */
char *psz_devicepath = char *psz_devicepath =
...@@ -526,18 +528,20 @@ void SPrefsPanel::apply() ...@@ -526,18 +528,20 @@ void SPrefsPanel::apply()
#endif #endif
//CaCi( "dv-caching" ) too short... //CaCi( "dv-caching" ) too short...
} }
break;
} }
/* Interfaces */ /* Interfaces */
if( number == SPrefsInterface ) case SPrefsInterface:
{ {
if( qobject_cast<QRadioButton *>(optionWidgets[skinRB])->isChecked() ) if( qobject_cast<QRadioButton *>(optionWidgets[skinRB])->isChecked() )
config_PutPsz( p_intf, "intf", "skins2" ); config_PutPsz( p_intf, "intf", "skins2" );
if( qobject_cast<QRadioButton *>(optionWidgets[qtRB])->isChecked() ) if( qobject_cast<QRadioButton *>(optionWidgets[qtRB])->isChecked() )
config_PutPsz( p_intf, "intf", "qt4" ); config_PutPsz( p_intf, "intf", "qt4" );
break;
} }
if( number == SPrefsAudio ) case SPrefsAudio:
{ {
bool b_normChecked = bool b_normChecked =
qobject_cast<QCheckBox *>(optionWidgets[normalizerChB])->isChecked(); qobject_cast<QCheckBox *>(optionWidgets[normalizerChB])->isChecked();
...@@ -564,6 +568,8 @@ void SPrefsPanel::apply() ...@@ -564,6 +568,8 @@ void SPrefsPanel::apply()
if( b_normChecked ) qs_filter.append( ":volnorm" ); if( b_normChecked ) qs_filter.append( ":volnorm" );
} }
config_PutPsz( p_intf, "audio-filter", qtu( qs_filter ) ); config_PutPsz( p_intf, "audio-filter", qtu( qs_filter ) );
break;
}
} }
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment