[vlc-devel] commit: Qt: Kill more warnings. (Jean-Baptiste Kempf )

git version control git at videolan.org
Fri Jan 16 16:34:44 CET 2009


vlc | branch: master | Jean-Baptiste Kempf <jb at videolan.org> | Wed Jan 14 18:20:16 2009 +0100| [f6081e574edd09f4fda05d461eec23bb50e6d2c2] | committer: Jean-Baptiste Kempf 

Qt: Kill more warnings.

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=f6081e574edd09f4fda05d461eec23bb50e6d2c2
---

 .../gui/qt4/components/complete_preferences.cpp    |    4 +++-
 modules/gui/qt4/components/controller_widget.cpp   |   10 ++++++++--
 modules/gui/qt4/components/extended_panels.cpp     |    1 +
 modules/gui/qt4/components/extended_panels.hpp     |    3 +--
 modules/gui/qt4/components/open_panels.cpp         |    3 +++
 modules/gui/qt4/components/open_panels.hpp         |    3 ---
 modules/gui/qt4/dialogs/vlm.cpp                    |    8 ++++++++
 modules/gui/qt4/main_interface.cpp                 |    1 +
 modules/gui/qt4/qt4.cpp                            |   14 +-------------
 9 files changed, 26 insertions(+), 21 deletions(-)

diff --git a/modules/gui/qt4/components/complete_preferences.cpp b/modules/gui/qt4/components/complete_preferences.cpp
index 9120ff6..1b520be 100644
--- a/modules/gui/qt4/components/complete_preferences.cpp
+++ b/modules/gui/qt4/components/complete_preferences.cpp
@@ -201,7 +201,9 @@ PrefsTree::PrefsTree( intf_thread_t *_p_intf, QWidget *_parent ) :
         // Main module excluded
         if( module_is_main( p_module) ) continue;
 
-        unsigned i_subcategory = 0, i_category = 0, confsize;
+        unsigned  confsize;
+        int i_subcategory = 0, i_category = 0;
+
         bool b_options = false;
         module_config_t *const p_config = module_config_get (p_module, &confsize);
 
diff --git a/modules/gui/qt4/components/controller_widget.cpp b/modules/gui/qt4/components/controller_widget.cpp
index 688bb3d..01d8e4e 100644
--- a/modules/gui/qt4/components/controller_widget.cpp
+++ b/modules/gui/qt4/components/controller_widget.cpp
@@ -38,7 +38,7 @@
 
 SoundWidget::SoundWidget( QWidget *_parent, intf_thread_t * _p_intf,
                           bool b_shiny )
-                         : b_my_volume( false ), QWidget( _parent )
+                         : QWidget( _parent ), b_my_volume( false )
 {
     p_intf = _p_intf;
     QHBoxLayout *layout = new QHBoxLayout( this );
@@ -159,14 +159,20 @@ void AtoB_Button::setIcons( bool timeA, bool timeB )
 
 bool VolumeClickHandler::eventFilter( QObject *obj, QEvent *e )
 {
+    VLC_UNUSED( obj );
     if (e->type() == QEvent::MouseButtonPress  )
     {
         aout_VolumeMute( p_intf, NULL );
         audio_volume_t i_volume;
         aout_VolumeGet( p_intf, &i_volume );
 //        m->updateVolume( i_volume *  VOLUME_MAX / (AOUT_VOLUME_MAX/2) );
+        e->accept();
         return true;
     }
-    return false;
+    else
+    {
+        e->ignore();
+        return false;
+    }
 }
 
diff --git a/modules/gui/qt4/components/extended_panels.cpp b/modules/gui/qt4/components/extended_panels.cpp
index e85b948..93a73bd 100644
--- a/modules/gui/qt4/components/extended_panels.cpp
+++ b/modules/gui/qt4/components/extended_panels.cpp
@@ -39,6 +39,7 @@
 #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>
diff --git a/modules/gui/qt4/components/extended_panels.hpp b/modules/gui/qt4/components/extended_panels.hpp
index 8951ab6..3d498c7 100644
--- a/modules/gui/qt4/components/extended_panels.hpp
+++ b/modules/gui/qt4/components/extended_panels.hpp
@@ -35,11 +35,10 @@
 #include "ui/equalizer.h"
 #include "ui/video_effects.h"
 #include "ui/v4l2.h"
-#include "../../audio_filter/equalizer_presets.h"
 
 #include <QTabWidget>
 
-#define BANDS EQZ_BANDS_MAX
+#define BANDS 10
 #define NUM_SP_CTRL 5
 
 class QSignalMapper;
diff --git a/modules/gui/qt4/components/open_panels.cpp b/modules/gui/qt4/components/open_panels.cpp
index a183712..1ea3c58 100644
--- a/modules/gui/qt4/components/open_panels.cpp
+++ b/modules/gui/qt4/components/open_panels.cpp
@@ -49,6 +49,9 @@
 
 #define I_DEVICE_TOOLTIP N_("Select the device or the VIDEO_TS directory")
 
+static const char *psz_devModule[] = { "v4l", "v4l2", "pvr", "dvb", "bda",
+                                       "dshow", "screen", "jack" };
+
 /**************************************************************************
  * Open Files and subtitles                                               *
  **************************************************************************/
diff --git a/modules/gui/qt4/components/open_panels.hpp b/modules/gui/qt4/components/open_panels.hpp
index 6b34a14..deaf898 100644
--- a/modules/gui/qt4/components/open_panels.hpp
+++ b/modules/gui/qt4/components/open_panels.hpp
@@ -71,9 +71,6 @@ enum
     JACK_DEVICE
 };
 
-static const char *psz_devModule[] = { "v4l", "v4l2", "pvr", "dvb", "bda",
-                                       "dshow", "screen", "jack" };
-
 class QWidget;
 class QLineEdit;
 class QString;
diff --git a/modules/gui/qt4/dialogs/vlm.cpp b/modules/gui/qt4/dialogs/vlm.cpp
index a297072..dc3b4c9 100644
--- a/modules/gui/qt4/dialogs/vlm.cpp
+++ b/modules/gui/qt4/dialogs/vlm.cpp
@@ -816,12 +816,20 @@ void VLMWrapper::EditSchedule( const QString name, const QString input,
         _schetime.toString( "hh:mm:ss" ) + "\"";
     vlm_ExecuteCommand( p_vlm, qtu( command ), &message );
     vlm_MessageDelete( message );
+
     if( _scherepeatnumber > 0 )
     {
        command = "setup \"" + name + "\" repeat \"" + _scherepeatnumber + "\"";
        vlm_ExecuteCommand( p_vlm, qtu( command ), &message );
        vlm_MessageDelete( message );
     }
+
+    if( _repeatDays > 0 )
+    {
+       command = "setup \"" + name + "\" period \"" + _repeatDays + "\"";
+       vlm_ExecuteCommand( p_vlm, qtu( command ), &message );
+       vlm_MessageDelete( message );
+    }
 }
 
 
diff --git a/modules/gui/qt4/main_interface.cpp b/modules/gui/qt4/main_interface.cpp
index fd02b67..fcbde2a 100644
--- a/modules/gui/qt4/main_interface.cpp
+++ b/modules/gui/qt4/main_interface.cpp
@@ -640,6 +640,7 @@ void MainInterface::releaseVideoSlot( void )
 /* Call from WindowControl function */
 int MainInterface::controlVideo( void *p_window, int i_query, va_list args )
 {
+    VLC_UNUSED( p_window ); //FIXME remove this param
     int i_ret = VLC_SUCCESS;
     switch( i_query )
     {
diff --git a/modules/gui/qt4/qt4.cpp b/modules/gui/qt4/qt4.cpp
index f1216da..d36de79 100644
--- a/modules/gui/qt4/qt4.cpp
+++ b/modules/gui/qt4/qt4.cpp
@@ -471,23 +471,11 @@ static void *Thread( void *obj )
 static void ShowDialog( intf_thread_t *p_intf, int i_dialog_event, int i_arg,
                         intf_dialog_args_t *p_arg )
 {
+    VLC_UNUSED( p_intf );
     DialogEvent *event = new DialogEvent( i_dialog_event, i_arg, p_arg );
     QApplication::postEvent( THEDP, static_cast<QEvent*>(event) );
 }
 
-/*****************************************************************************
- * PopupMenuCB: callback to show the popupmenu.
- *  We don't show the menu directly here because we don't want the
- *  caller to block for a too long time.
- *****************************************************************************/
-static int PopupMenuCB( vlc_object_t *p_this, const char *psz_variable,
-                        vlc_value_t old_val, vlc_value_t new_val, void *param )
-{
-    intf_thread_t *p_intf = (intf_thread_t *)param;
-    ShowDialog( p_intf, INTF_DIALOG_POPUPMENU, new_val.b_bool, 0 );
-    return VLC_SUCCESS;
-}
-
 /**
  * Video output window provider
  */




More information about the vlc-devel mailing list