]> git.sesse.net Git - vlc/blobdiff - modules/gui/qt4/components/extended_panels.cpp
Fix potential memleak.
[vlc] / modules / gui / qt4 / components / extended_panels.cpp
old mode 100755 (executable)
new mode 100644 (file)
index 783b8fa..018a697
 #include "qt4.hpp"
 #include "input_manager.hpp"
 
-#include "../../audio_filter/equalizer_presets.h"
 #include <vlc_aout.h>
 #include <vlc_intf_strings.h>
 #include <vlc_vout.h>
 #include <vlc_osd.h>
 
-#include <iostream>
-#include <string.h>
 
 #if 0
 class ConfClickHandler : public QObject
@@ -186,8 +183,8 @@ ExtVideo::ExtVideo( intf_thread_t *_p_intf, QTabWidget *_parent ) :
     SETUP_VFILTER_OPTION( wallColsSpin, valueChanged( int ) )
 
     SETUP_VFILTER( panoramix )
-    SETUP_VFILTER_OPTION( panorRowsSpin, valueChanged( int ) )
-    SETUP_VFILTER_OPTION( panorColsSpin, valueChanged( int ) )
+    SETUP_VFILTER_OPTION( panoramixRowsSpin, valueChanged( int ) )
+    SETUP_VFILTER_OPTION( panoramixColsSpin, valueChanged( int ) )
 
 
     SETUP_VFILTER( erase )
@@ -207,15 +204,36 @@ ExtVideo::ExtVideo( intf_thread_t *_p_intf, QTabWidget *_parent ) :
 
 #undef SETUP_VFILTER
 #undef SETUP_VFILTER_OPTION
+
+    CONNECT( ui.cropTopPx, valueChanged( int ), this, cropChange() );
+    CONNECT( ui.cropBotPx, valueChanged( int ), this, cropChange() );
+    CONNECT( ui.cropLeftPx, valueChanged( int ), this, cropChange() );
+    CONNECT( ui.cropRightPx, valueChanged( int ), this, cropChange() );
+    CONNECT( ui.topBotCropSync, toggled( bool ),
+             ui.cropBotPx, setDisabled( bool ) );
+    CONNECT( ui.leftRightCropSync, toggled( bool ),
+             ui.cropRightPx, setDisabled( bool ) );
 }
 
 ExtVideo::~ExtVideo()
 {
 }
 
-void ExtVideo::ChangeVFiltersString( char *psz_name, vlc_bool_t b_add )
+void ExtVideo::cropChange()
+{
+    p_vout = ( vout_thread_t * )vlc_object_find( p_intf,
+                                VLC_OBJECT_VOUT, FIND_CHILD );
+    if( p_vout )
+    {
+        var_SetInteger( p_vout, "crop-top", ui.cropTopPx->value() );
+        var_SetInteger( p_vout, "crop-bottom", ui.cropBotPx->value() );
+        var_SetInteger( p_vout, "crop-left", ui.cropLeftPx->value() );
+        var_SetInteger( p_vout, "crop-right", ui.cropRightPx->value() );
+    }
+}
+
+void ExtVideo::ChangeVFiltersString( char *psz_name, bool b_add )
 {
-    vout_thread_t *p_vout;
     char *psz_parser, *psz_string;
     const char *psz_filter_type;
 
@@ -261,7 +279,10 @@ void ExtVideo::ChangeVFiltersString( char *psz_name, vlc_bool_t b_add )
             psz_parser = psz_string;
             if( asprintf( &psz_string, ( *psz_string ) ? "%s:%s" : "%s%s",
                             psz_string, psz_name ) == -1 )
+            {
+                free( psz_parser );
                 return;
+            }
             free( psz_parser );
         }
         else
@@ -425,7 +446,7 @@ void ExtVideo::setWidgetValue( QObject *widget )
         else if( lineedit )
         {
             char str[30];
-            sprintf( str, "%06X", val.i_int );
+            snprintf( str, sizeof(str), "%06X", val.i_int );
             lineedit->setText( str );
         }
         else if( combobox ) combobox->setCurrentIndex(
@@ -605,7 +626,6 @@ void ExtV4l2::Refresh( void )
     }
     if( p_obj )
     {
-        msg_Dbg( p_intf, "Found v4l2 instance" );
         vlc_value_t val, text, name;
         int i_ret = var_Change( p_obj, "controls", VLC_VAR_GETCHOICES,
                                 &val, &text );
@@ -729,7 +749,6 @@ void ExtV4l2::Refresh( void )
         msg_Dbg( p_intf, "Couldn't find v4l2 instance" );
         ui.help->show();
     }
-
 }
 
 void ExtV4l2::ValueChange( bool value )
@@ -789,22 +808,15 @@ Equalizer::Equalizer( intf_thread_t *_p_intf, QWidget *_parent ) :
     smallFont.setPointSize( smallFont.pointSize() - 3 );
 
     ui.setupUi( this );
-    presetsComboBox = ui.presetsCombo;
-
     ui.preampLabel->setFont( smallFont );
-    ui.preampSlider->setMaximum( 400 );
-    for( int i = 0 ; i < NB_PRESETS ; i ++ )
-    {
-        ui.presetsCombo->addItem( qtr( preset_list_text[i] ),
-                                  QVariant( i ) );
-    }
-    CONNECT( ui.presetsCombo, activated( int ), this, setPreset( int ) );
 
-    BUTTONACT( ui.enableCheck, enable() );
-    BUTTONACT( ui.eq2PassCheck, set2Pass() );
-
-    CONNECT( ui.preampSlider, valueChanged( int ), this, setPreamp() );
+    /* Setup of presetsComboBox */
+    presetsComboBox = ui.presetsCombo;
+    CONNECT( presetsComboBox, currentIndexChanged( int ),
+             this, updateUISliderValues( int ) );
+    CONNECT( presetsComboBox, activated( int ), this, setCorePreset( int ) );
 
+    /* Add the sliders for the Bands */
     QGridLayout *grid = new QGridLayout( ui.frame );
     grid->setMargin( 0 );
     for( int i = 0 ; i < BANDS ; i++ )
@@ -812,26 +824,53 @@ Equalizer::Equalizer( intf_thread_t *_p_intf, QWidget *_parent ) :
         bands[i] = new QSlider( Qt::Vertical );
         bands[i]->setMaximum( 400 );
         bands[i]->setValue( 200 );
-        CONNECT( bands[i], valueChanged( int ), this, setBand() );
+        CONNECT( bands[i], valueChanged( int ), this, setCoreBands() );
+
         band_texts[i] = new QLabel( band_frequencies[i] + "\n0.0dB" );
         band_texts[i]->setFont( smallFont );
+
         grid->addWidget( bands[i], 0, i );
         grid->addWidget( band_texts[i], 1, i );
     }
 
-    /* Write down initial values */
+    /* Add the listed presets */
+    for( int i = 0 ; i < NB_PRESETS ; i ++ )
+    {
+        presetsComboBox->addItem( qtr( preset_list_text[i] ),
+                                  QVariant( preset_list[i] ) );
+    }
+
+    /* Connects */
+    BUTTONACT( ui.enableCheck, enable() );
+    BUTTONACT( ui.eq2PassCheck, set2Pass() );
+    CONNECT( ui.preampSlider, valueChanged( int ), this, setPreamp() );
+
+    /* Do the update from the value of the core */
+    updateUIFromCore();
+}
+
+Equalizer::~Equalizer()
+{
+}
+
+/* Write down initial values */
+void Equalizer::updateUIFromCore()
+{
+    char *psz_af;//, *psz_bands; Don't use it ?
+    float f_preamp;
+    int i_preset;
+
     aout_instance_t *p_aout = ( aout_instance_t * )vlc_object_find( p_intf,
                                     VLC_OBJECT_AOUT, FIND_ANYWHERE );
-    char *psz_af;
-    char *psz_bands;
-    float f_preamp;
     if( p_aout )
     {
         psz_af = var_GetNonEmptyString( p_aout, "audio-filter" );
         if( var_GetBool( p_aout, "equalizer-2pass" ) )
             ui.eq2PassCheck->setChecked( true );
-        psz_bands = var_GetNonEmptyString( p_aout, "equalizer-bands" );
+//        psz_bands = var_GetNonEmptyString( p_aout, "equalizer-bands" );
         f_preamp = var_GetFloat( p_aout, "equalizer-preamp" );
+        i_preset = presetsComboBox->findData( QVariant(
+                    var_GetString( p_aout, "equalizer-preset" ) ) );
         vlc_object_release( p_aout );
     }
     else
@@ -839,36 +878,42 @@ Equalizer::Equalizer( intf_thread_t *_p_intf, QWidget *_parent ) :
         psz_af = config_GetPsz( p_intf, "audio-filter" );
         if( config_GetInt( p_intf, "equalizer-2pass" ) )
             ui.eq2PassCheck->setChecked( true );
-        psz_bands = config_GetPsz( p_intf, "equalizer-bands" );
+//        psz_bands = config_GetPsz( p_intf, "equalizer-bands" );
         f_preamp = config_GetFloat( p_intf, "equalizer-preamp" );
+        i_preset = presetsComboBox->findData( QVariant(
+                    config_GetPsz( p_intf, "equalizer-preset" ) ) );
     }
     if( psz_af && strstr( psz_af, "equalizer" ) != NULL )
         ui.enableCheck->setChecked( true );
-    free( psz_af );
     enable( ui.enableCheck->isChecked() );
 
-    setValues( psz_bands, f_preamp );
-}
+    presetsComboBox->setCurrentIndex( i_preset );
 
-Equalizer::~Equalizer()
-{
+    free( psz_af );
 }
 
+/* Functin called when enableButton is toggled */
 void Equalizer::enable()
 {
     bool en = ui.enableCheck->isChecked();
     aout_EnableFilter( VLC_OBJECT( p_intf ), "equalizer",
-                       en ? VLC_TRUE : VLC_FALSE );
+                       en ? true : false );
 //    aout_EnableFilter( VLC_OBJECT( p_intf ), "upmixer",
-//                       en ? VLC_TRUE : VLC_FALSE );
+//                       en ? true : false );
 //     aout_EnableFilter( VLC_OBJECT( p_intf ), "vsurround",
-//                       en ? VLC_TRUE : VLC_FALSE );
-     enable( en );
+//                       en ? true : false );
+    enable( en );
+
+    if( presetsComboBox->currentIndex() < 0 )
+        presetsComboBox->setCurrentIndex( 0 );
+
 }
 
 void Equalizer::enable( bool en )
 {
     ui.eq2PassCheck->setEnabled( en );
+    presetsComboBox->setEnabled( en );
+    ui.presetLabel->setEnabled( en );
     ui.preampLabel->setEnabled( en );
     ui.preampSlider->setEnabled( en  );
     for( int i = 0 ; i< BANDS; i++ )
@@ -877,11 +922,12 @@ void Equalizer::enable( bool en )
     }
 }
 
+/* Function called when the set2Pass button is activated */
 void Equalizer::set2Pass()
 {
     aout_instance_t *p_aout= ( aout_instance_t * )vlc_object_find( p_intf,
                                  VLC_OBJECT_AOUT, FIND_ANYWHERE );
-    vlc_bool_t b_2p = ui.eq2PassCheck->isChecked();
+    bool b_2p = ui.eq2PassCheck->isChecked();
 
     if( p_aout == NULL )
         config_PutInt( p_intf, "equalizer-2pass", b_2p );
@@ -891,105 +937,126 @@ void Equalizer::set2Pass()
         config_PutInt( p_intf, "equalizer-2pass", b_2p );
         for( int i = 0; i < p_aout->i_nb_inputs; i++ )
         {
-            p_aout->pp_inputs[i]->b_restart = VLC_TRUE;
+            p_aout->pp_inputs[i]->b_restart = true;
         }
         vlc_object_release( p_aout );
     }
 }
 
+/* Function called when the preamp slider is moved */
 void Equalizer::setPreamp()
 {
-    float f= ( float )(  ui.preampSlider->value() ) /10 - 20;
-    char psz_val[5];
+    const float f = ( float )(  ui.preampSlider->value() ) /10 - 20;
     aout_instance_t *p_aout= ( aout_instance_t * )vlc_object_find( p_intf,
                                        VLC_OBJECT_AOUT, FIND_ANYWHERE );
 
-    sprintf( psz_val, "%.1f", f );
-    ui.preampLabel->setText( qtr( "Preamp\n" ) + psz_val + qtr( "dB" ) );
+    ui.preampLabel->setText( qtr( "Preamp\n" ) + QString::number( f, 'f', 1 )
+                                               + qtr( "dB" ) );
     if( p_aout )
     {
-        delCallbacks( p_aout );
+        //delCallbacks( p_aout );
         var_SetFloat( p_aout, "equalizer-preamp", f );
-        addCallbacks( p_aout );
+        //addCallbacks( p_aout );
         vlc_object_release( p_aout );
     }
     config_PutFloat( p_intf, "equalizer-preamp", f );
 }
 
-void Equalizer::setBand()
+void Equalizer::setCoreBands()
 {
-    char psz_values[102]; memset( psz_values, 0, 102 );
-
     /**\todo smoothing */
 
-    for( int i = 0 ; i< BANDS ; i++ )
+    QString values;
+    for( int i = 0; i < BANDS; i++ )
     {
-        char psz_val[5];
-        float f_val = ( float )(  bands[i]->value() ) / 10 - 20 ;
-        sprintf( psz_values, "%s %f", psz_values, f_val );
-        sprintf( psz_val, "% 5.1f", f_val );
-        band_texts[i]->setText( band_frequencies[i] + "\n" + psz_val + "dB" );
+        const float f_val = (float)( bands[i]->value() ) / 10 - 20;
+        QString val = QString("%1").arg( f_val, 5, 'f', 1 );
+
+        band_texts[i]->setText( band_frequencies[i] + "\n" + val + "dB" );
+        values += " " + val;
     }
+    const char *psz_values = values.toAscii().constData();
+
     aout_instance_t *p_aout= ( aout_instance_t * )vlc_object_find( p_intf,
                                           VLC_OBJECT_AOUT, FIND_ANYWHERE );
     if( p_aout )
     {
-        delCallbacks( p_aout );
+        //delCallbacks( p_aout );
         var_SetString( p_aout, "equalizer-bands", psz_values );
-        addCallbacks( p_aout );
+        //addCallbacks( p_aout );
         vlc_object_release( p_aout );
     }
 }
-void Equalizer::setValues( char *psz_bands, float f_preamp )
+
+void Equalizer::updateUISliderValues( int i_preset )
 {
-    char *p = psz_bands;
+    if( i_preset < 0 ) return;
+
+    char *p = createValuesFromPreset( i_preset );
+    char *psz = p;
+    float f_preamp = eqz_preset_10b[i_preset]->f_preamp;
+
     if ( p )
     {
-        for( int i = 0; i < 10; i++ )
+        for( int i = 0; i < BANDS; i++ )
         {
-            char psz_val[5];
-            float f = strtof( p, &p );
-            int  i_val= ( int )( ( f + 20 ) * 10 );
-            bands[i]->setValue(  i_val );
-            sprintf( psz_val, "% 5.1f", f );
-            band_texts[i]->setText( band_frequencies[i] + "\n" + psz_val +
-                                    "dB" );
-            if( p == NULL || *p == '\0' ) break;
+            const float f = us_strtod(p, &p );
+
+            bands[i]->setValue( (int)( ( f + 20 ) * 10 )  );
+
+            band_texts[i]->setText( band_frequencies[i] + "\n" 
+                                  + QString("%1").arg( f, 5, 'f', 1 ) + "dB" );
+            if( p == NULL || *p == '\0' )
+                break;
             p++;
-            if( *p == '\0' )  break;
+            if( *p == '\0' )
+                break;
         }
+        free( psz );
     }
-    char psz_val[5];
-    int i_val = ( int )( ( f_preamp + 20 ) * 10 );
-    sprintf( psz_val, "%.1f", f_preamp );
-    ui.preampSlider->setValue( i_val );
-    ui.preampLabel->setText( qtr( "Preamp\n" ) + psz_val + qtr( "dB" ) );
+    ui.preampSlider->setValue( (int)( ( f_preamp + 20 ) * 10 ) );
+    ui.preampLabel->setText( qtr( "Preamp\n" )
+                   + QString::number( f_preamp, 'f', 1 ) + qtr( "dB" ) );
 }
 
-void Equalizer::setPreset( int preset )
+char * Equalizer::createValuesFromPreset( int i_preset )
 {
-    aout_instance_t *p_aout= ( aout_instance_t * )vlc_object_find( p_intf,
-                                                VLC_OBJECT_AOUT, FIND_ANYWHERE );
+    char *psz_values;
+    QString values;
+
+    /* Create the QString in Qt */
+    for( int i = 0 ; i< BANDS ;i++ )
+        values += QString( " %1" ).arg( eqz_preset_10b[i_preset]->f_amp[i] );
 
-    char psz_values[102]; memset( psz_values, 0, 102 );
-    for( int i = 0 ; i< 10 ;i++ )
-        sprintf( psz_values, "%s %.1f", psz_values,
-                                        eqz_preset_10b[preset]->f_amp[i] );
+    /* Convert it to char * */
+    if( !asprintf( &psz_values, "%s", values.toAscii().constData() ) )
+        return NULL;
 
+    return psz_values;
+}
+
+void Equalizer::setCorePreset( int i_preset )
+{
+    char *psz_values = createValuesFromPreset( i_preset );
+    if( !psz_values ) return ;
+
+    aout_instance_t *p_aout= ( aout_instance_t * )vlc_object_find( p_intf,
+                                               VLC_OBJECT_AOUT, FIND_ANYWHERE );
     if( p_aout )
     {
         delCallbacks( p_aout );
+        var_SetString( p_aout , "equalizer-preset" , preset_list[i_preset] );
+
         var_SetString( p_aout, "equalizer-bands", psz_values );
         var_SetFloat( p_aout, "equalizer-preamp",
-                      eqz_preset_10b[preset]->f_preamp );
+                      eqz_preset_10b[i_preset]->f_preamp );
         addCallbacks( p_aout );
         vlc_object_release( p_aout );
     }
     config_PutPsz( p_intf, "equalizer-bands", psz_values );
+    config_PutPsz( p_intf, "equalizer-preset", preset_list[i_preset] );
     config_PutFloat( p_intf, "equalizer-preamp",
-                    eqz_preset_10b[preset]->f_preamp );
-
-    setValues( psz_values, eqz_preset_10b[preset]->f_preamp );
+                    eqz_preset_10b[i_preset]->f_preamp );
 }
 
 static int PresetCallback( vlc_object_t *p_this, char const *psz_cmd,
@@ -997,7 +1064,8 @@ static int PresetCallback( vlc_object_t *p_this, char const *psz_cmd,
 {
     char *psz_preset = newval.psz_string;
     Equalizer *eq = ( Equalizer * )p_data;
-    eq->presetsComboBox->setCurrentIndex( eq->presetsComboBox->findText( qfu( psz_preset ) ) );
+    int i_preset = eq->presetsComboBox->findData( QVariant( psz_preset ) );
+    eq->presetsComboBox->setCurrentIndex( i_preset );
     return VLC_SUCCESS;
 }
 
@@ -1036,7 +1104,7 @@ Spatializer::Spatializer( intf_thread_t *_p_intf, QWidget *_parent ) :
     QGridLayout *layout = new QGridLayout( this );
     layout->setMargin( 0 );
 
-    enableCheck = new QCheckBox( qfu( "Enable spatializer" ) );
+    enableCheck = new QCheckBox( qtr( "Enable spatializer" ) );
     layout->addWidget( enableCheck, 0, 0, 1, NUM_SP_CTRL );
 
     for( int i = 0 ; i < NUM_SP_CTRL ; i++ )
@@ -1103,7 +1171,7 @@ void Spatializer::enable()
 {
     bool en = enableCheck->isChecked();
     aout_EnableFilter( VLC_OBJECT( p_intf ), "spatializer",
-            en ? VLC_TRUE : VLC_FALSE );
+            en ? true : false );
     enable( en );
 }
 
@@ -1123,16 +1191,13 @@ void Spatializer::setInitValues()
 
 void Spatializer::setValues( float *controlVars )
 {
-    char psz_val[5];
-    char var_name[5];
     aout_instance_t *p_aout= ( aout_instance_t * )
         vlc_object_find( p_intf, VLC_OBJECT_AOUT, FIND_ANYWHERE );
 
     for( int i = 0 ; i < NUM_SP_CTRL ; i++ )
     {
-        float f= ( float )(  spatCtrl[i]->value() );
-        sprintf( psz_val, "%.1f", f );
-        ctrl_readout[i]->setText( psz_val );
+        float f = (float)(  spatCtrl[i]->value() );
+        ctrl_readout[i]->setText( QString::number( f, 'f',  1 ) );
     }
     if( p_aout )
     {
@@ -1272,8 +1337,6 @@ SyncControls::SyncControls( intf_thread_t *_p_intf, QWidget *_parent ) :
 
     updateButton = new QToolButton;
     updateButton->setAutoRaise( true );
-    updateButton->setText( "u" );
-    updateButton->setToolTip( qtr( "Force update of the values in this dialog" ) );
     mainLayout->addWidget( updateButton, 0, 4, 1, 1 );
 
 
@@ -1288,7 +1351,8 @@ SyncControls::SyncControls( intf_thread_t *_p_intf, QWidget *_parent ) :
     CONNECT( subsSpin, valueChanged ( double ), this, advanceSubs( double ) ) ;
     CONNECT( subSpeedSpin, valueChanged ( double ),
              this, adjustSubsSpeed( double ) );
-    BUTTONACT( updateButton, update() );
+    BUTTON_SET_ACT_I( updateButton, "", update,
+            qtr( "Force update of this dialog's values" ), update() );
 
     /* Set it */
     update();