skins2: kill many compil warnings

diff --git a/modules/gui/skins2/commands/cmd_callbacks.hpp b/modules/gui/skins2/commands/cmd_callbacks.hpp
index 76cf23decdff4ee23478d6faf3f4a4f40bf4972e..e3f3cfb0395c9dbbe3b1e509ad83d47951202d1d 100644 (file)
--- a/modules/gui/skins2/commands/cmd_callbacks.hpp
+++ b/modules/gui/skins2/commands/cmd_callbacks.hpp
@@ -36,7 +36,7 @@ public:
                  void (VlcProc::*func)(vlc_object_t *,vlc_value_t),
                  string label )
         : CmdGeneric( pIntf ), m_pObj( pObj ), m_newVal( newVal ),
-          m_pfExecute( func ), m_label( label )
+          m_label( label ), m_pfExecute( func )
     {
         if( m_pObj )
             vlc_object_hold( m_pObj );

diff --git a/modules/gui/skins2/controls/ctrl_button.cpp b/modules/gui/skins2/controls/ctrl_button.cpp
index 9aca8d8a5dedb9e061e2d09634e669182a4663e1..0530bbbefeadf5d451cb2cc5f2dfdcd944bd7af8 100644 (file)
--- a/modules/gui/skins2/controls/ctrl_button.cpp
+++ b/modules/gui/skins2/controls/ctrl_button.cpp
@@ -161,6 +161,7 @@ void CtrlButton::setImage( AnimBitmap *pImg )
 
 void CtrlButton::onUpdate( Subject<AnimBitmap> &rBitmap, void *arg )
 {
+    (void)rBitmap;(void)arg;
     notifyLayout( m_pImg->getWidth(), m_pImg->getHeight() );
 }
 

diff --git a/modules/gui/skins2/controls/ctrl_checkbox.cpp b/modules/gui/skins2/controls/ctrl_checkbox.cpp
index bd56de02e7774e2b5b796cb21f2cbfb2313fff8a..31f35feec59ebdcd720e58215cb9021d089cbc5d 100644 (file)
--- a/modules/gui/skins2/controls/ctrl_checkbox.cpp
+++ b/modules/gui/skins2/controls/ctrl_checkbox.cpp
@@ -246,12 +246,14 @@ void CtrlCheckbox::CmdHiddenUp::execute()
 
 void CtrlCheckbox::onVarBoolUpdate( VarBool &rVariable )
 {
+    (void)rVariable;
     changeButton();
 }
 
 
 void CtrlCheckbox::onUpdate( Subject<AnimBitmap> &rBitmap, void *arg )
 {
+    (void)rBitmap;(void)arg;
     notifyLayout( m_pImgCurrent->getWidth(), m_pImgCurrent->getHeight() );
 }
 

diff --git a/modules/gui/skins2/controls/ctrl_generic.cpp b/modules/gui/skins2/controls/ctrl_generic.cpp
index d17fe7ba6e46be2bd56d3dfc400d2cba5e345862..7bded8c3b29bb64679013df125ba8a2b8ef0a608 100644 (file)
--- a/modules/gui/skins2/controls/ctrl_generic.cpp
+++ b/modules/gui/skins2/controls/ctrl_generic.cpp
@@ -164,6 +164,7 @@ bool CtrlGeneric::isVisible() const
 
 void CtrlGeneric::onUpdate( Subject<VarBool> &rVariable, void *arg  )
 {
+    (void)arg;
     // Is it the visibility variable ?
     if( &rVariable == m_pVisible )
     {

diff --git a/modules/gui/skins2/controls/ctrl_generic.hpp b/modules/gui/skins2/controls/ctrl_generic.hpp
index cc33f4620a1b3e1f8cd8d659cde4b5a9c8f464b8..ee7afeb8df5c53ca9faa3d543cd8a3997d96a776 100644 (file)
--- a/modules/gui/skins2/controls/ctrl_generic.hpp
+++ b/modules/gui/skins2/controls/ctrl_generic.hpp
@@ -47,13 +47,15 @@ public:
     virtual ~CtrlGeneric();
 
     /// Handle an event on the control
-    virtual void handleEvent( EvtGeneric &rEvent ) { }
+    virtual void handleEvent( EvtGeneric &rEvent ) { (void)rEvent; }
 
     /// Check whether coordinates are inside the control
-    virtual bool mouseOver( int x, int y ) const { return false; }
+    virtual bool mouseOver( int x, int y ) const
+        { (void)x; (void)y; return false; }
 
     /// Draw the control on the given graphics
-    virtual void draw( OSGraphics &rImage, int xDest, int yDest, int w, int h ) { }
+    virtual void draw( OSGraphics &rImage, int xDest,
+                       int yDest, int w, int h ) = 0;
 
     /// Set the position and the associated layout of the control
     virtual void setLayout( GenericLayout *pLayout,
@@ -124,7 +126,7 @@ protected:
     virtual void onPositionChange() { }
 
     /// Overload this method to get notified of bool variable changes
-    virtual void onVarBoolUpdate( VarBool &rVar ) { }
+    virtual void onVarBoolUpdate( VarBool &rVar ) { (void)rVar; }
 
     /// Method called when an observed bool variable is changed
     virtual void onUpdate( Subject<VarBool> &rVariable , void* );

diff --git a/modules/gui/skins2/controls/ctrl_image.cpp b/modules/gui/skins2/controls/ctrl_image.cpp
index 9709de7e9ff9284d7fcecba4db9885d01abd3c06..cb60e3f095707397f2ee92e5e739c0fdff1dab5a 100644 (file)
--- a/modules/gui/skins2/controls/ctrl_image.cpp
+++ b/modules/gui/skins2/controls/ctrl_image.cpp
@@ -240,6 +240,7 @@ void CtrlImage::draw( OSGraphics &rImage, int xDest, int yDest, int w, int h )
 
 void CtrlImage::onUpdate( Subject<VarString> &rVariable, void* arg )
 {
+    (void)arg;
     VlcProc *pVlcProc = VlcProc::instance( getIntf() );
 
     if( &rVariable == &pVlcProc->getStreamArtVar() )

diff --git a/modules/gui/skins2/controls/ctrl_list.cpp b/modules/gui/skins2/controls/ctrl_list.cpp
index 0ff0732f0805e3858e0cf9dd7e8e091a23a49839..015c7b86d253f43f73e8b9ac20e620a5b29957eb 100644 (file)
--- a/modules/gui/skins2/controls/ctrl_list.cpp
+++ b/modules/gui/skins2/controls/ctrl_list.cpp
@@ -68,6 +68,7 @@ CtrlList::~CtrlList()
 
 void CtrlList::onUpdate( Subject<VarList> &rList, void *arg  )
 {
+    (void)rList; (void)arg;
     autoScroll();
     m_pLastSelected = NULL;
 }
@@ -75,6 +76,7 @@ void CtrlList::onUpdate( Subject<VarList> &rList, void *arg  )
 
 void CtrlList::onUpdate( Subject<VarPercent> &rPercent, void *arg  )
 {
+    (void)rPercent; (void)arg;
     // Get the size of the control
     const Position *pPos = getPosition();
     if( !pPos )

diff --git a/modules/gui/skins2/controls/ctrl_radialslider.cpp b/modules/gui/skins2/controls/ctrl_radialslider.cpp
index 6ed798fd5b2d198b25fa42cfd1f9d28179d8b5fb..14a6e2d274b532e1c7ddb90b772b73c1e0cc477c 100644 (file)
--- a/modules/gui/skins2/controls/ctrl_radialslider.cpp
+++ b/modules/gui/skins2/controls/ctrl_radialslider.cpp
@@ -105,9 +105,9 @@ void CtrlRadialSlider::draw( OSGraphics &rImage, int xDest, int yDest, int w, in
 }
 
 
-void CtrlRadialSlider::onUpdate( Subject<VarPercent> &rVariable,
-                                 void *arg  )
+void CtrlRadialSlider::onUpdate( Subject<VarPercent> &rVariable, void *arg )
 {
+    (void)arg;
     if( &rVariable == &m_rVariable )
     {
         int position = (int)( m_rVariable.get() * ( m_numImg - 1 ) );

diff --git a/modules/gui/skins2/controls/ctrl_slider.cpp b/modules/gui/skins2/controls/ctrl_slider.cpp
index 1198bc5af5cc7b5ce777b7169c113f156d075490..caa97f9cf42c56537b013e079dcefa2a053a5613 100644 (file)
--- a/modules/gui/skins2/controls/ctrl_slider.cpp
+++ b/modules/gui/skins2/controls/ctrl_slider.cpp
@@ -205,9 +205,9 @@ void CtrlSliderCursor::notifyLayout( int width, int height, int xOffSet, int yOf
 }
 
 
-void CtrlSliderCursor::onUpdate( Subject<VarPercent> &rVariable,
-                                 void *arg  )
+void CtrlSliderCursor::onUpdate( Subject<VarPercent> &rVariable, void *arg  )
 {
+    (void)rVariable; (void)arg;
     // The position has changed
     refreshLayout( false );
 }
@@ -518,6 +518,7 @@ void CtrlSliderBg::associateCursor( CtrlSliderCursor &rCursor )
 
 void CtrlSliderBg::onUpdate( Subject<VarPercent> &rVariable, void*arg )
 {
+    (void)rVariable; (void)arg;
     int position = (int)( m_rVariable.get() * (m_nbHoriz * m_nbVert - 1) );
     if( position == m_position )
         return;

diff --git a/modules/gui/skins2/controls/ctrl_text.cpp b/modules/gui/skins2/controls/ctrl_text.cpp
index cfbdf6100815a26c9cc13416a5d1d2aa9fe6115d..61d41ae97e42f741950a6525aad951d39ca3d531 100644 (file)
--- a/modules/gui/skins2/controls/ctrl_text.cpp
+++ b/modules/gui/skins2/controls/ctrl_text.cpp
@@ -215,6 +215,7 @@ void CtrlText::setText( const UString &rText, uint32_t color )
 
 void CtrlText::onUpdate( Subject<VarText> &rVariable, void* arg )
 {
+    (void)rVariable; (void)arg;
     if( isVisible() )
     {
         displayText( m_rVariable.get() );
@@ -225,6 +226,7 @@ void CtrlText::onUpdate( Subject<VarText> &rVariable, void* arg )
 
 void CtrlText::onUpdate( Subject<VarBool> &rVariable, void *arg  )
 {
+    (void)arg;
     // Visibility changed
     if( &rVariable == m_pVisible )
     {

diff --git a/modules/gui/skins2/controls/ctrl_tree.cpp b/modules/gui/skins2/controls/ctrl_tree.cpp
index d71d042af71c3b5637a04ef12b1aa567813cc681..48d694d3bd1ad3b73716dea3cd0639cdb0842146 100644 (file)
--- a/modules/gui/skins2/controls/ctrl_tree.cpp
+++ b/modules/gui/skins2/controls/ctrl_tree.cpp
@@ -59,10 +59,10 @@ CtrlTree::CtrlTree( intf_thread_t *pIntf,
     CtrlGeneric( pIntf,rHelp, pVisible), m_rTree( rTree), m_rFont( rFont ),
     m_pBgBitmap( pBgBitmap ), m_pItemBitmap( pItemBitmap ),
     m_pOpenBitmap( pOpenBitmap ), m_pClosedBitmap( pClosedBitmap ),
+    m_pScaledBitmap( NULL ),
     m_fgColor( fgColor ), m_playColor( playColor ), m_bgColor1( bgColor1 ),
     m_bgColor2( bgColor2 ), m_selColor( selColor ),
-    m_pLastSelected( NULL ), m_pImage( NULL ), m_pScaledBitmap( NULL ),
-    m_dontMove( false )
+    m_pLastSelected( NULL ), m_pImage( NULL ), m_dontMove( false )
 {
     // Observe the tree and position variables
     m_rTree.addObserver( this );
@@ -140,6 +140,7 @@ int CtrlTree::maxItems()
 void CtrlTree::onUpdate( Subject<VarTree, tree_update> &rTree,
                          tree_update *arg )
 {
+    (void)rTree;
     if( arg->type == arg->UpdateItem ) // Item update
     {
         if( arg->b_active_item )
@@ -196,6 +197,7 @@ void CtrlTree::onUpdate( Subject<VarTree, tree_update> &rTree,
 
 void CtrlTree::onUpdate( Subject<VarPercent> &rPercent, void* arg)
 {
+    (void)rPercent; (void)arg;
     // Determine what is the first item to display
     VarTree::Iterator it = m_flat ? m_rTree.firstLeaf() : m_rTree.begin();
 
@@ -712,7 +714,6 @@ bool CtrlTree::ensureVisible( VarTree::Iterator item )
 void CtrlTree::autoScroll()
 {
     // Find the current playing stream
-    int playIndex = 0;
     VarTree::Iterator it;
 
     for( it = m_flat ? m_rTree.firstLeaf() : m_rTree.begin();

diff --git a/modules/gui/skins2/controls/ctrl_video.cpp b/modules/gui/skins2/controls/ctrl_video.cpp
index 163c5f627217849706c080923cd11145128994a6..2ed026bdee14a3055fe3a546d56e209c02564d91 100644 (file)
--- a/modules/gui/skins2/controls/ctrl_video.cpp
+++ b/modules/gui/skins2/controls/ctrl_video.cpp
@@ -57,11 +57,13 @@ CtrlVideo::~CtrlVideo()
 
 void CtrlVideo::handleEvent( EvtGeneric &rEvent )
 {
+    (void)rEvent;
 }
 
 
 bool CtrlVideo::mouseOver( int x, int y ) const
 {
+    (void)x; (void)y;
     return false;
 }
 
@@ -161,6 +163,8 @@ void CtrlVideo::resizeControl( int width, int height )
 
 void CtrlVideo::onUpdate( Subject<VarBool> &rVariable, void *arg  )
 {
+    (void)arg;
+
     // Visibility changed
     if( &rVariable == m_pVisible )
     {

diff --git a/modules/gui/skins2/parser/builder.cpp b/modules/gui/skins2/parser/builder.cpp
index 686005581c1f51502559f5909ef98bbe7ccd631d..e5e7dc2ebd297dcff80c11f3ff7ce1625770f19f 100644 (file)
--- a/modules/gui/skins2/parser/builder.cpp
+++ b/modules/gui/skins2/parser/builder.cpp
@@ -1193,7 +1193,7 @@ string Builder::getFilePath( const string &rFileName ) const
         // For skins to be valid on both Linux and Win32,
         // slash should be used as path separator for both OSs.
         msg_Warn( getIntf(), "use of '/' is preferred to '\\' for paths" );
-        int pos;
+        string::size_type pos;
         while( ( pos = file.find( "\\" ) ) != string::npos )
            file[pos] = '/';
     }

diff --git a/modules/gui/skins2/parser/interpreter.cpp b/modules/gui/skins2/parser/interpreter.cpp
index 9f695aafffff9a9a2e9fe4d1c5d82f5cdf3675a9..75f75302a770648dda4ee2e1fcf06a0d340dbbcc 100644 (file)
--- a/modules/gui/skins2/parser/interpreter.cpp
+++ b/modules/gui/skins2/parser/interpreter.cpp
@@ -507,6 +507,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme )
 
 VarPercent *Interpreter::getVarPercent( const string &rName, Theme *pTheme )
 {
+    (void)pTheme;
     VarManager *pVarManager = VarManager::instance( getIntf() );
     return static_cast<VarPercent*>(pVarManager->getVar( rName, "percent" ));
 }
@@ -514,6 +515,7 @@ VarPercent *Interpreter::getVarPercent( const string &rName, Theme *pTheme )
 
 VarList *Interpreter::getVarList( const string &rName, Theme *pTheme )
 {
+    (void)pTheme;
     VarManager *pVarManager = VarManager::instance( getIntf() );
     return static_cast<VarList*>(pVarManager->getVar( rName, "list" ));
 }
@@ -521,6 +523,7 @@ VarList *Interpreter::getVarList( const string &rName, Theme *pTheme )
 
 VarTree *Interpreter::getVarTree( const string &rName, Theme *pTheme )
 {
+    (void)pTheme;
     VarManager *pVarManager = VarManager::instance( getIntf() );
     return static_cast<VarTree*>(pVarManager->getVar( rName, "tree" ));
 }

diff --git a/modules/gui/skins2/parser/skin_parser.cpp b/modules/gui/skins2/parser/skin_parser.cpp
index c699778b2438ccbcff81166621a18628327929b6..30ed199f6ff5738bec19066fe262b02baad70c27 100644 (file)
--- a/modules/gui/skins2/parser/skin_parser.cpp
+++ b/modules/gui/skins2/parser/skin_parser.cpp
@@ -871,7 +871,7 @@ const string SkinParser::uniqueId( const string &id )
     return newId;
 }
 
-const int SkinParser::getRefWidth( bool toScreen )
+int SkinParser::getRefWidth( bool toScreen )
 {
     if( toScreen )
     {
@@ -892,7 +892,7 @@ const int SkinParser::getRefWidth( bool toScreen )
     }
 }
 
-const int SkinParser::getRefHeight( bool toScreen )
+int SkinParser::getRefHeight( bool toScreen )
 {
     if( toScreen )
     {
@@ -914,11 +914,11 @@ const int SkinParser::getRefHeight( bool toScreen )
 }
 
 
-const int SkinParser::getDimension( string value, int refDimension )
+int SkinParser::getDimension( string value, int refDimension )
 {
-    int val;
+    string::size_type leftPos;
 
-    int leftPos = value.find( "%" );
+    leftPos = value.find( "%" );
     if( leftPos != string::npos )
     {
         int val = atoi( value.substr( 0, leftPos ).c_str() );
@@ -928,7 +928,7 @@ const int SkinParser::getDimension( string value, int refDimension )
     leftPos = value.find( "px" );
     if( leftPos != string::npos )
     {
-        int val  = atoi( value.substr( 0, leftPos ).c_str() );
+        int val = atoi( value.substr( 0, leftPos ).c_str() );
         return val;
     }
 
@@ -936,7 +936,7 @@ const int SkinParser::getDimension( string value, int refDimension )
 }
 
 
-const int SkinParser::getPosition( string position )
+int SkinParser::getPosition( string position )
 {
     if( position == "-1" )
         return POS_UNDEF;

diff --git a/modules/gui/skins2/parser/skin_parser.hpp b/modules/gui/skins2/parser/skin_parser.hpp
index 77e6ba26edc89ee569f962cd440624c291fc8484..f9d955097983f4c695c550c3c276a2b9e854118d 100644 (file)
--- a/modules/gui/skins2/parser/skin_parser.hpp
+++ b/modules/gui/skins2/parser/skin_parser.hpp
@@ -98,10 +98,10 @@ private:
     const string uniqueId( const string &id );
 
     /// Management of relative positions
-    const int getRefWidth( bool toScreen );
-    const int getRefHeight( bool toScreen );
-    const int getDimension( string value, int refDimension );
-    const int getPosition( string value );
+    int getRefWidth( bool toScreen );
+    int getRefHeight( bool toScreen );
+    int getDimension( string value, int refDimension );
+    int getPosition( string value );
     void updateWindowPos( int width, int height );
 
     void convertPosition( string position,

diff --git a/modules/gui/skins2/parser/xmlparser.hpp b/modules/gui/skins2/parser/xmlparser.hpp
index 4645bd790868bf631d8928cd3f5d5c2df0ebb2e4..19c30b70496393b17db8bb7347fbdd5b08908c04 100644 (file)
--- a/modules/gui/skins2/parser/xmlparser.hpp
+++ b/modules/gui/skins2/parser/xmlparser.hpp
@@ -59,9 +59,9 @@ protected:
     bool m_errors;
 
     /// Callbacks
-    virtual void handleBeginElement( const string &rName,
-                                     AttrList_t &attr ) { }
-    virtual void handleEndElement( const string &rName ) { }
+    virtual void handleBeginElement( const string &rName, AttrList_t &attr )
+        { (void)rName; (void)attr; }
+    virtual void handleEndElement( const string &rName ) { (void)rName; }
 
 private:
     void LoadCatalog();

diff --git a/modules/gui/skins2/src/anim_bitmap.cpp b/modules/gui/skins2/src/anim_bitmap.cpp
index 32880183edd42f4af9224387e140b9df9bacb572..92a844f640ea6b0a0e0f3981d17ebbcb090f7ce4 100644 (file)
--- a/modules/gui/skins2/src/anim_bitmap.cpp
+++ b/modules/gui/skins2/src/anim_bitmap.cpp
@@ -29,8 +29,8 @@
 
 
 AnimBitmap::AnimBitmap( intf_thread_t *pIntf, const GenericBitmap &rBitmap ):
-    SkinObject( pIntf ), m_pImage( NULL ), m_curFrame( 0 ), m_curLoop( 0 ),
-    m_pTimer( NULL ), m_cmdNextFrame( this ), m_rBitmap( rBitmap )
+    SkinObject( pIntf ), m_rBitmap( rBitmap ), m_pImage( NULL ),
+    m_curFrame( 0 ), m_curLoop( 0 ), m_pTimer( NULL ), m_cmdNextFrame( this )
 {
     // Build the graphics
     OSFactory *pOsFactory = OSFactory::instance( pIntf );

diff --git a/modules/gui/skins2/src/bitmap_font.cpp b/modules/gui/skins2/src/bitmap_font.cpp
index 3a9bd295f28390fab73ce93bba64b1a9dbcf9f61..f7594c8e77e3ed009ac408eba8f021e1ccfa0866 100644 (file)
--- a/modules/gui/skins2/src/bitmap_font.cpp
+++ b/modules/gui/skins2/src/bitmap_font.cpp
@@ -82,6 +82,8 @@ BitmapFont::BitmapFont( intf_thread_t *pIntf, const GenericBitmap &rBitmap,
 GenericBitmap *BitmapFont::drawString( const UString &rString,
                                        uint32_t color, int maxWidth ) const
 {
+    (void)color; (void)maxWidth;
+
     uint32_t *pString = (uint32_t*)rString.u_str();
     // Compute the text width
     int width = 0;

diff --git a/modules/gui/skins2/src/dialogs.cpp b/modules/gui/skins2/src/dialogs.cpp
index f3aac6fe715a0738cdc941fcc769b176386728ec..ccfcf9ed1075e86b2f49b127d9baef4ffefcd18d 100644 (file)
--- a/modules/gui/skins2/src/dialogs.cpp
+++ b/modules/gui/skins2/src/dialogs.cpp
@@ -96,6 +96,8 @@ void Dialogs::showPlaylistSaveCB( intf_dialog_args_t *pArg )
 static int PopupMenuCB( vlc_object_t *p_this, const char *psz_variable,
                         vlc_value_t old_val, vlc_value_t new_val, void *param )
 {
+    (void)p_this; (void)psz_variable; (void)old_val;
+
     Dialogs *p_dialogs = (Dialogs *)param;
     p_dialogs->showPopupMenu( new_val.b_bool != 0, INTF_DIALOG_POPUPMENU );
 

diff --git a/modules/gui/skins2/src/file_bitmap.cpp b/modules/gui/skins2/src/file_bitmap.cpp
index 051de8a693dba47122afd34b72bf94f992433cda..d7ed27aac70d5184c6f0a7bdfaa5f92ff7c24237 100644 (file)
--- a/modules/gui/skins2/src/file_bitmap.cpp
+++ b/modules/gui/skins2/src/file_bitmap.cpp
@@ -37,10 +37,11 @@ FileBitmap::FileBitmap( intf_thread_t *pIntf, image_handler_t *pImageHandler,
     GenericBitmap( pIntf, nbFrames, fps, nbLoops ), m_width( 0 ), m_height( 0 ),
     m_pData( NULL )
 {
-    video_format_t fmt_in = {0}, fmt_out = {0};
+    video_format_t fmt_in, fmt_out;
     picture_t *pPic;
 
-    fmt_out.i_chroma = VLC_CODEC_RGBA;
+    video_format_Init( &fmt_in, 0 );
+    video_format_Init( &fmt_out, VLC_CODEC_RGBA );
 
     char* psz_uri = make_URI( fileName.c_str(), NULL );
     pPic = image_ReadUrl( pImageHandler, psz_uri, &fmt_in, &fmt_out );

diff --git a/modules/gui/skins2/src/fsc_window.cpp b/modules/gui/skins2/src/fsc_window.cpp
index 5c9051d1fbfb31454cd6efbb9d33e58dd2d65c3f..d490f67611ab4538424f649d94f45de612c21424 100644 (file)
--- a/modules/gui/skins2/src/fsc_window.cpp
+++ b/modules/gui/skins2/src/fsc_window.cpp
@@ -44,9 +44,11 @@ FscWindow::FscWindow( intf_thread_t *pIntf, int left, int top,
                       WindowManager &rWindowManager,
                       bool dragDrop, bool playOnDrop, bool visible ) :
     TopWindow( pIntf, left, top, rWindowManager, dragDrop,
-               playOnDrop, false, GenericWindow::FscWindow ), m_cmdFscHide( this ),
-               m_opacity( m_opacity ), m_count( 0 )
+               playOnDrop, false, GenericWindow::FscWindow ),
+    m_pTimer( NULL ), m_count( 0 ), m_opacity( m_opacity ),
+    m_cmdFscHide( this )
 {
+    (void)visible;
     m_pTimer = OSFactory::instance( getIntf() )->createOSTimer( m_cmdFscHide );
 
     VarBool &rFullscreen = VlcProc::instance( getIntf() )->getFullscreenVar();

diff --git a/modules/gui/skins2/src/generic_window.cpp b/modules/gui/skins2/src/generic_window.cpp
index 1e95707d78ad90dd34a21d7ac2c3b862065658c3..fce0b2a4fbd8de1f7f3ddaf6161d7edb7760b893 100644 (file)
--- a/modules/gui/skins2/src/generic_window.cpp
+++ b/modules/gui/skins2/src/generic_window.cpp
@@ -132,8 +132,9 @@ void GenericWindow::toggleOnTop( bool onTop ) const
 }
 
 
-void GenericWindow::onUpdate( Subject<VarBool> &rVariable, void*arg )
+void GenericWindow::onUpdate( Subject<VarBool> &rVariable, void* arg )
 {
+    (void)rVariable; (void)arg;
     if (&rVariable == m_pVarVisible )
     {
         if( m_pVarVisible->get() )

diff --git a/modules/gui/skins2/src/generic_window.hpp b/modules/gui/skins2/src/generic_window.hpp
index ee4a8e33304c4f8e85fba9ad14f98cf88f7ed4b6..2bc5ebac0bcfecad024967f7aed6cce8b35a2e5b 100644 (file)
--- a/modules/gui/skins2/src/generic_window.hpp
+++ b/modules/gui/skins2/src/generic_window.hpp
@@ -65,13 +65,13 @@ public:
     virtual ~GenericWindow();
 
     /// Methods to process OS events.
-    virtual void processEvent( EvtFocus &rEvtFocus ) { }
-    virtual void processEvent( EvtMenu &rEvtMenu ) { }
-    virtual void processEvent( EvtMotion &rEvtMotion ) { }
-    virtual void processEvent( EvtMouse &rEvtMouse ) { }
-    virtual void processEvent( EvtLeave &rEvtLeave ) { }
-    virtual void processEvent( EvtKey &rEvtKey ) { }
-    virtual void processEvent( EvtScroll &rEvtScroll ) { }
+    virtual void processEvent( EvtFocus &rEvtFocus ) { (void)rEvtFocus; }
+    virtual void processEvent( EvtMenu &rEvtMenu ) { (void)rEvtMenu; }
+    virtual void processEvent( EvtMotion &rEvtMotion ) { (void)rEvtMotion; }
+    virtual void processEvent( EvtMouse &rEvtMouse ) { (void)rEvtMouse; }
+    virtual void processEvent( EvtLeave &rEvtLeave ) { (void)rEvtLeave; }
+    virtual void processEvent( EvtKey &rEvtKey ) { (void)rEvtKey; }
+    virtual void processEvent( EvtScroll &rEvtScroll ) { (void)rEvtScroll; }
 
     virtual void processEvent( EvtRefresh &rEvtRefresh );
 
@@ -79,7 +79,8 @@ public:
     virtual void resize( int width, int height );
 
     /// Refresh an area of the window
-    virtual void refresh( int left, int top, int width, int height ) { }
+    virtual void refresh( int left, int top, int width, int height )
+        { (void)left; (void)top; (void)width; (void)height; }
 
     /// Invalidate an area of the window
     virtual void invalidateRect( int left, int top, int width, int height );

diff --git a/modules/gui/skins2/src/skin_main.cpp b/modules/gui/skins2/src/skin_main.cpp
index 9497303b1317b6502a0199cd77e2afc19f64e6d3..f9ae23d2a51f41b2d1670fc220a7f3c81ce00ae4 100644 (file)
--- a/modules/gui/skins2/src/skin_main.cpp
+++ b/modules/gui/skins2/src/skin_main.cpp
@@ -404,7 +404,6 @@ static int WindowControl( vout_window_t *pWnd, int query, va_list args )
 {
     vout_window_sys_t* sys = pWnd->sys;
     intf_thread_t *pIntf = sys->pIntf;
-    VoutManager *pVoutManager = VoutManager::instance( pIntf );
     AsyncQueue *pQueue = AsyncQueue::instance( pIntf );
 
     switch( query )

diff --git a/modules/gui/skins2/src/theme_loader.cpp b/modules/gui/skins2/src/theme_loader.cpp
index b976ff745d9be5cce331265c4e3162f1265a82cb..b6553fe5bdfdb1a5211cfa60cc16fe9f153dd095 100644 (file)
--- a/modules/gui/skins2/src/theme_loader.cpp
+++ b/modules/gui/skins2/src/theme_loader.cpp
@@ -493,6 +493,8 @@ union tar_buffer {
 
 int tar_open( TAR **t, char *pathname, int oflags )
 {
+    (void)oflags;
+
     gzFile f = gzopen( pathname, "rb" );
     if( f == NULL )
     {
@@ -713,6 +715,8 @@ static void * currentGzVp = NULL;
 
 int gzopen_frontend( const char *pathname, int oflags, int mode )
 {
+    (void)mode;
+
     const char *gzflags;
     gzFile gzf;
 

diff --git a/modules/gui/skins2/src/theme_repository.cpp b/modules/gui/skins2/src/theme_repository.cpp
index a4a53abc8458d925849f56eae65469f2db8ed99e..27951485ab6bce73d7ebbdc89994360274d46f42 100644 (file)
--- a/modules/gui/skins2/src/theme_repository.cpp
+++ b/modules/gui/skins2/src/theme_repository.cpp
@@ -138,7 +138,6 @@ void ThemeRepository::parseDirectory( const string &rDir_locale )
 {
     DIR *pDir;
     char *pszDirContent;
-    vlc_value_t val, text;
     // Path separator
     const string &sep = OSFactory::instance( getIntf() )->getDirSeparator();
 
@@ -167,7 +166,7 @@ void ThemeRepository::parseDirectory( const string &rDir_locale )
         {
             string path = rDir + sep + name;
             string shortname = name.substr( 0, name.size() - 4 );
-            for( int i = 0; i < shortname.size(); i++ )
+            for( string::size_type i = 0; i < shortname.size(); i++ )
                 shortname[i] = ( i == 0 ) ?
                                toupper( shortname[i] ) :
                                tolower( shortname[i] );
@@ -188,6 +187,7 @@ int ThemeRepository::changeSkin( vlc_object_t *pIntf, char const *pVariable,
                                  vlc_value_t oldval, vlc_value_t newval,
                                  void *pData )
 {
+    (void)pIntf; (void)oldval;
     ThemeRepository *pThis = (ThemeRepository*)(pData);
 
     if( !strcmp( pVariable, "intf-skins-interactive" ) )

diff --git a/modules/gui/skins2/src/tooltip.cpp b/modules/gui/skins2/src/tooltip.cpp
index 644ef7950d14590bff8bdbbbfad0249e37149810..20888c8d2532f41119623f5c4011ec9e20777ec9 100644 (file)
--- a/modules/gui/skins2/src/tooltip.cpp
+++ b/modules/gui/skins2/src/tooltip.cpp
@@ -71,8 +71,9 @@ void Tooltip::hide()
 }
 
 
-void Tooltip::onUpdate( Subject<VarText> &rVariable , void *arg)
+void Tooltip::onUpdate( Subject<VarText> &rVariable, void *arg )
 {
+    (void)arg;
     // Redisplay the tooltip
     displayText( ((VarText&)rVariable).get() );
 }

diff --git a/modules/gui/skins2/src/top_window.cpp b/modules/gui/skins2/src/top_window.cpp
index 908a6d30764a4baa0185cc008a6ff95620fa8741..60dc2a977a5497be4a233a6b171cb7e69ec17b2b 100644 (file)
--- a/modules/gui/skins2/src/top_window.cpp
+++ b/modules/gui/skins2/src/top_window.cpp
@@ -75,7 +75,7 @@ TopWindow::~TopWindow()
 
 void TopWindow::processEvent( EvtFocus &rEvtFocus )
 {
-//    fprintf(stderr, rEvtFocus.getAsString().c_str());
+    (void)rEvtFocus;
 }
 
 
@@ -130,9 +130,10 @@ void TopWindow::processEvent( EvtMotion &rEvtMotion )
 
 void TopWindow::processEvent( EvtLeave &rEvtLeave )
 {
+    (void)rEvtLeave;
+
     // No more hit control
     setLastHit( NULL );
-
     if( !m_pCapturingControl )
     {
         m_rWindowManager.hideTooltip();

diff --git a/modules/gui/skins2/src/vlcproc.cpp b/modules/gui/skins2/src/vlcproc.cpp
index 5528fd7a8ca6215663ef0656d82f3a918495d67c..bad8ad96e126458b88e809c52db61f4df4b7385c 100644 (file)
--- a/modules/gui/skins2/src/vlcproc.cpp
+++ b/modules/gui/skins2/src/vlcproc.cpp
@@ -178,7 +178,6 @@ VlcProc::VlcProc( intf_thread_t *pIntf ): SkinObject( pIntf ),
     // Called when we have an interaction dialog to display
     var_Create( pIntf, "interaction", VLC_VAR_ADDRESS );
     var_AddCallback( pIntf, "interaction", onInteraction, this );
-    interaction_Register( pIntf );
 
     // initialize variables refering to liblvc and playlist objects
     init_variables();
@@ -253,6 +252,7 @@ void VlcProc::CmdManage::execute()
 int VlcProc::onInputNew( vlc_object_t *pObj, const char *pVariable,
                          vlc_value_t oldval, vlc_value_t newval, void *pParam )
 {
+    (void)pObj; (void)pVariable; (void)oldval;
     VlcProc *pThis = (VlcProc*)pParam;
     input_thread_t *pInput = static_cast<input_thread_t*>(newval.p_address);
 
@@ -269,6 +269,7 @@ int VlcProc::onItemChange( vlc_object_t *pObj, const char *pVariable,
                            vlc_value_t oldval, vlc_value_t newval,
                            void *pParam )
 {
+    (void)pObj; (void)pVariable; (void)oldval;
     VlcProc *pThis = (VlcProc*)pParam;
     input_item_t *p_item = static_cast<input_item_t*>(newval.p_address);
 
@@ -287,6 +288,7 @@ int VlcProc::onItemAppend( vlc_object_t *pObj, const char *pVariable,
                            vlc_value_t oldVal, vlc_value_t newVal,
                            void *pParam )
 {
+    (void)pObj; (void)pVariable; (void)oldVal;
     VlcProc *pThis = (VlcProc*)pParam;
 
     playlist_add_t *p_add = static_cast<playlist_add_t*>(newVal.p_address);
@@ -304,6 +306,7 @@ int VlcProc::onItemDelete( vlc_object_t *pObj, const char *pVariable,
                            vlc_value_t oldVal, vlc_value_t newVal,
                            void *pParam )
 {
+    (void)pObj; (void)pVariable; (void)oldVal;
     VlcProc *pThis = (VlcProc*)pParam;
 
     int i_id = newVal.i_int;
@@ -321,6 +324,7 @@ int VlcProc::onInteraction( vlc_object_t *pObj, const char *pVariable,
                             vlc_value_t oldVal, vlc_value_t newVal,
                             void *pParam )
 {
+    (void)pObj; (void)pVariable; (void)oldVal;
     VlcProc *pThis = (VlcProc*)pParam;
     interaction_dialog_t *p_dialog = (interaction_dialog_t *)(newVal.p_address);
 
@@ -334,6 +338,7 @@ int VlcProc::onEqBandsChange( vlc_object_t *pObj, const char *pVariable,
                               vlc_value_t oldVal, vlc_value_t newVal,
                               void *pParam )
 {
+    (void)pObj; (void)pVariable; (void)oldVal;
     VlcProc *pThis = (VlcProc*)pParam;
 
     // Post a set equalizer bands command
@@ -351,6 +356,7 @@ int VlcProc::onEqPreampChange( vlc_object_t *pObj, const char *pVariable,
                                vlc_value_t oldVal, vlc_value_t newVal,
                                void *pParam )
 {
+    (void)pObj; (void)pVariable; (void)oldVal;
     VlcProc *pThis = (VlcProc*)pParam;
     EqualizerPreamp *pVarPreamp = (EqualizerPreamp*)(pThis->m_cVarEqPreamp.get());
 
@@ -368,6 +374,7 @@ int VlcProc::onGenericCallback( vlc_object_t *pObj, const char *pVariable,
                                 vlc_value_t oldVal, vlc_value_t newVal,
                                 void *pParam )
 {
+    (void)oldVal;
     VlcProc *pThis = (VlcProc*)pParam;
     AsyncQueue *pQueue = AsyncQueue::instance( pThis->getIntf() );
 
@@ -412,6 +419,7 @@ int VlcProc::onGenericCallback2( vlc_object_t *pObj, const char *pVariable,
                                  vlc_value_t oldVal, vlc_value_t newVal,
                                  void *pParam )
 {
+    (void)oldVal;
     VlcProc *pThis = (VlcProc*)pParam;
     AsyncQueue *pQueue = AsyncQueue::instance( pThis->getIntf() );
 
@@ -467,6 +475,7 @@ int VlcProc::onGenericCallback2( vlc_object_t *pObj, const char *pVariable,
 
 void VlcProc::on_item_current_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 {
+    (void)p_obj;
     input_item_t *p_item = static_cast<input_item_t*>(newVal.p_address);
 
     // Update short name
@@ -641,6 +650,7 @@ void VlcProc::on_intf_event_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 
 void VlcProc::on_bit_rate_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 {
+    (void)newVal;
     input_thread_t* pInput = (input_thread_t*) p_obj;
 
     assert( getIntf()->p_sys->p_input == NULL || getIntf()->p_sys->p_input == pInput );
@@ -651,6 +661,7 @@ void VlcProc::on_bit_rate_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 
 void VlcProc::on_sample_rate_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 {
+    (void)newVal;
     input_thread_t* pInput = (input_thread_t*) p_obj;
 
     assert( getIntf()->p_sys->p_input == NULL || getIntf()->p_sys->p_input == pInput );
@@ -661,6 +672,7 @@ void VlcProc::on_sample_rate_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 
 void VlcProc::on_can_record_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 {
+    (void)newVal;
     input_thread_t* pInput = (input_thread_t*) p_obj;
 
     assert( getIntf()->p_sys->p_input == NULL || getIntf()->p_sys->p_input == pInput );
@@ -670,6 +682,7 @@ void VlcProc::on_can_record_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 
 void VlcProc::on_random_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 {
+    (void)newVal;
     playlist_t* pPlaylist = (playlist_t*) p_obj;
 
     SET_BOOL( m_cVarRandom, var_GetBool( pPlaylist, "random" ) );
@@ -677,6 +690,7 @@ void VlcProc::on_random_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 
 void VlcProc::on_loop_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 {
+    (void)newVal;
     playlist_t* pPlaylist = (playlist_t*) p_obj;
 
     SET_BOOL( m_cVarLoop, var_GetBool( pPlaylist, "loop" ) );
@@ -684,6 +698,7 @@ void VlcProc::on_loop_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 
 void VlcProc::on_repeat_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 {
+    (void)newVal;
     playlist_t* pPlaylist = (playlist_t*) p_obj;
 
     SET_BOOL( m_cVarRepeat, var_GetBool( pPlaylist, "repeat" ) );
@@ -701,6 +716,7 @@ void VlcProc::on_volume_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 
 void VlcProc::on_audio_filter_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 {
+    (void)newVal;
     aout_instance_t* pAout = (aout_instance_t*) p_obj;
 
     char *pFilters = newVal.psz_string;
@@ -717,7 +733,7 @@ void VlcProc::on_audio_filter_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 
 void VlcProc::on_intf_show_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 {
-    (void)p_obj;
+    (void)p_obj; (void)newVal;
     bool b_fullscreen = getFullscreenVar().get();
 
     if( !b_fullscreen )
@@ -762,6 +778,7 @@ void VlcProc::on_intf_show_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 
 void VlcProc::on_mouse_moved_changed( vlc_object_t* p_obj, vlc_value_t newVal )
 {
+    (void)p_obj; (void)newVal;
     FscWindow* pFscWindow = VoutManager::instance( getIntf() )->getFscWindow();
     if( pFscWindow )
         pFscWindow->onMouseMoved();

diff --git a/modules/gui/skins2/src/vout_manager.cpp b/modules/gui/skins2/src/vout_manager.cpp
index 88718f990b3d1f5a8a8205854dd4ba035a151172..7ba61c5b5af361d29102a192a51e70c39886484d 100644 (file)
--- a/modules/gui/skins2/src/vout_manager.cpp
+++ b/modules/gui/skins2/src/vout_manager.cpp
@@ -50,8 +50,8 @@ void VoutManager::destroy( intf_thread_t *pIntf )
 
 
 VoutManager::VoutManager( intf_thread_t *pIntf ): SkinObject( pIntf ),
-     m_pVoutMainWindow( NULL ), m_pFscWindow( NULL ), m_pCtrlVideoVec(),
-     m_pCtrlVideoVecBackup(), m_SavedWndVec()
+     m_pCtrlVideoVec(), m_pCtrlVideoVecBackup(), m_SavedWndVec(),
+     m_pVoutMainWindow( NULL ), m_pFscWindow( NULL )
 {
     m_pVoutMainWindow = new VoutMainWindow( getIntf() );
 
@@ -281,8 +281,9 @@ void VoutManager::setFullscreenWnd( vout_window_t *pWnd, bool b_fullscreen )
 }
 
 
-void VoutManager::onUpdate( Subject<VarBool> &rVariable, void *arg  )
+void VoutManager::onUpdate( Subject<VarBool> &rVariable, void *arg )
 {
+    (void)arg;
     VarBool &rFullscreen = VlcProc::instance( getIntf() )->getFullscreenVar();
     if( &rVariable == &rFullscreen )
     {

diff --git a/modules/gui/skins2/src/vout_window.cpp b/modules/gui/skins2/src/vout_window.cpp
index 7ee460dfee07e22ef68c08156face587b20acd33..e2093633994f7ed1a6694914bfc31c467e300482 100644 (file)
--- a/modules/gui/skins2/src/vout_window.cpp
+++ b/modules/gui/skins2/src/vout_window.cpp
@@ -38,7 +38,7 @@ VoutWindow::VoutWindow( intf_thread_t *pIntf, vout_window_t* pWnd,
       GenericWindow( pIntf, 0, 0, false, false, pParent,
                      GenericWindow::VoutWindow ),
       m_pWnd( pWnd ), original_width( width ), original_height( height ),
-      m_pParentWindow( pParent ), m_pCtrlVideo( NULL )
+      m_pCtrlVideo( NULL ), m_pParentWindow( pParent )
 {
     if( m_pWnd )
     {
@@ -51,7 +51,6 @@ VoutWindow::VoutWindow( intf_thread_t *pIntf, vout_window_t* pWnd,
         m_pWnd->handle.hwnd = getOSHandle();
 #endif
     }
-
 }
 
 

diff --git a/modules/gui/skins2/src/window_manager.cpp b/modules/gui/skins2/src/window_manager.cpp
index e9efb22d563305b5e26b60c156d4d5bea9593669..b611e15cfa3e9f34acbb80734d01c843454833df 100644 (file)
--- a/modules/gui/skins2/src/window_manager.cpp
+++ b/modules/gui/skins2/src/window_manager.cpp
@@ -32,9 +32,9 @@
 
 
 WindowManager::WindowManager( intf_thread_t *pIntf ):
-    SkinObject( pIntf ), m_magnet( 0 ), m_direction( kNone ),
-    m_maximizeRect(0, 0, 50, 50), m_pTooltip( NULL ), m_pPopup( NULL ),
-    m_alpha( 255 ), m_moveAlpha( 255 ), m_opacityEnabled( false )
+    SkinObject( pIntf ), m_magnet( 0 ), m_alpha( 255 ), m_moveAlpha( 255 ),
+    m_opacityEnabled( false ), m_opacity( 255 ), m_direction( kNone ),
+    m_maximizeRect(0, 0, 50, 50), m_pTooltip( NULL ), m_pPopup( NULL )
 {
     // Create and register a variable for the "on top" status
     VarManager *pVarManager = VarManager::instance( getIntf() );
@@ -492,6 +492,7 @@ void WindowManager::buildDependSet( WinSet_t &rWinSet,
 void WindowManager::checkAnchors( TopWindow *pWindow,
                                   int &xOffset, int &yOffset ) const
 {
+    (void)pWindow;
     WinSet_t::const_iterator itMov, itSta;
     AncList_t::const_iterator itAncMov, itAncSta;
 

diff --git a/modules/gui/skins2/utils/var_bool.cpp b/modules/gui/skins2/utils/var_bool.cpp
index f5cb12cbaa10fecd60bcd7536446ea5c8227c07e..ebbaab8362dd551b5274ac5a62225a794d65fcf6 100644 (file)
--- a/modules/gui/skins2/utils/var_bool.cpp
+++ b/modules/gui/skins2/utils/var_bool.cpp
@@ -63,6 +63,7 @@ VarBoolAndBool::~VarBoolAndBool()
 
 void VarBoolAndBool::onUpdate( Subject<VarBool> &rVariable, void *arg )
 {
+    (void)rVariable; (void)arg;
     if( m_value != ( m_rVar1.get() && m_rVar2.get() ) )
     {
         m_value = ( m_rVar1.get() && m_rVar2.get() );
@@ -88,8 +89,9 @@ VarBoolOrBool::~VarBoolOrBool()
 }
 
 
-void VarBoolOrBool::onUpdate( Subject<VarBool> &rVariable , void*arg)
+void VarBoolOrBool::onUpdate( Subject<VarBool> &rVariable, void *arg )
 {
+    (void)rVariable; (void)arg;
     if( m_value != ( m_rVar1.get() || m_rVar2.get() ) )
     {
         m_value = ( m_rVar1.get() || m_rVar2.get() );
@@ -111,8 +113,9 @@ VarNotBool::~VarNotBool()
 }
 
 
-void VarNotBool::onUpdate( Subject<VarBool> &rVariable, void*arg )
+void VarNotBool::onUpdate( Subject<VarBool> &rVariable, void *arg )
 {
+    (void)rVariable; (void)arg;
     notify();
 }
 

diff --git a/modules/gui/skins2/utils/var_list.hpp b/modules/gui/skins2/utils/var_list.hpp
index 49d70e636a44482febc95c069472eb8160c757b6..7d3c681409e5f6a4651c50319788551081bd2af2 100644 (file)
--- a/modules/gui/skins2/utils/var_list.hpp
+++ b/modules/gui/skins2/utils/var_list.hpp
@@ -85,7 +85,7 @@ public:
     ConstIterator operator[]( int n ) const;
 
     /// Execute the action associated to this item
-    virtual void action( Elem_t *pItem ) { }
+    virtual void action( Elem_t *pItem ) { (void)pItem; }
 
     /// Get a reference on the position variable
     VarPercent &getPositionVar() const

diff --git a/modules/gui/skins2/utils/var_text.cpp b/modules/gui/skins2/utils/var_text.cpp
index 3be733f5993805cb339896fc8f2a08c6410bdb3f..18ff625da1873ced0a7b7f6343e0bd8be6112fcf 100644 (file)
--- a/modules/gui/skins2/utils/var_text.cpp
+++ b/modules/gui/skins2/utils/var_text.cpp
@@ -188,6 +188,7 @@ void VarText::set( const UString &rText )
 
 void VarText::onUpdate( Subject<VarPercent> &rVariable, void *arg )
 {
+    (void)rVariable; (void)arg;
     UString newText = get();
     // If the text has changed, notify the observers
     if( newText != m_lastText )
@@ -200,6 +201,7 @@ void VarText::onUpdate( Subject<VarPercent> &rVariable, void *arg )
 
 void VarText::onUpdate( Subject<VarText> &rVariable, void *arg )
 {
+    (void)rVariable; (void)arg;
     UString newText = get();
     // If the text has changed, notify the observers
     if( newText != m_lastText )

diff --git a/modules/gui/skins2/utils/var_tree.cpp b/modules/gui/skins2/utils/var_tree.cpp
index 2d867aff0516d611899a9bcff2ca9720c790123d..f2c0f87338e268c6ddb0dbd993a1f3d5521dd414 100644 (file)
--- a/modules/gui/skins2/utils/var_tree.cpp
+++ b/modules/gui/skins2/utils/var_tree.cpp
@@ -28,9 +28,9 @@
 const string VarTree::m_type = "tree";
 
 VarTree::VarTree( intf_thread_t *pIntf )
-    : Variable( pIntf ), m_id( 0 ), m_selected( false ), m_playing( false ),
-    m_expanded( false ), m_deleted( false ),
-    m_pData( NULL ), m_pParent( NULL ), m_readonly( false )
+    : Variable( pIntf ), m_pParent( NULL ), m_id( 0 ), m_pData( NULL ),
+      m_readonly( false ), m_selected( false ), m_playing( false ),
+      m_expanded( false ), m_deleted( false )
 {
     // Create the position variable
     m_cPosition = VariablePtr( new VarPercent( pIntf ) );
@@ -40,10 +40,10 @@ VarTree::VarTree( intf_thread_t *pIntf )
 VarTree::VarTree( intf_thread_t *pIntf, VarTree *pParent, int id,
                   const UStringPtr &rcString, bool selected, bool playing,
                   bool expanded, bool readonly, void *pData )
-    : Variable( pIntf ), m_id( id ), m_cString( rcString ),
-    m_selected( selected ), m_playing( playing ), m_expanded( expanded ),
-    m_deleted( false ), m_pData( pData ), m_pParent( pParent ),
-    m_readonly( readonly )
+    : Variable( pIntf ), m_pParent( pParent ),
+      m_id( id ), m_pData( pData ), m_cString( rcString ),
+      m_readonly( readonly ), m_selected( selected ), m_playing( playing ),
+      m_expanded( expanded ), m_deleted( false )
 {
     // Create the position variable
     m_cPosition = VariablePtr( new VarPercent( pIntf ) );

diff --git a/modules/gui/skins2/utils/var_tree.hpp b/modules/gui/skins2/utils/var_tree.hpp
index 0fadc1c82baf99f090f24dd0526eb02b97d634d0..5934c4fd1986daaf747cb3adf12b9bfa551bad76 100644 (file)
--- a/modules/gui/skins2/utils/var_tree.hpp
+++ b/modules/gui/skins2/utils/var_tree.hpp
@@ -131,7 +131,7 @@ public:
     void removeChild( Iterator it ) { m_children.erase( it ); }
 
     /// Execute the action associated to this item
-    virtual void action( VarTree *pItem ) { }
+    virtual void action( VarTree *pItem ) { (void)pItem; }
 
     /// Get a reference on the position variable
     VarPercent &getPositionVar() const

diff --git a/modules/gui/skins2/vars/equalizer.cpp b/modules/gui/skins2/vars/equalizer.cpp
index 38649d101c7e03db1b18aa7dc38a4688de0c38b9..40449e272649b6ec1fb5f33a7e0d3ad66d1109e0 100644 (file)
--- a/modules/gui/skins2/vars/equalizer.cpp
+++ b/modules/gui/skins2/vars/equalizer.cpp
@@ -83,6 +83,7 @@ VariablePtr EqualizerBands::getBand( int band )
 
 void EqualizerBands::onUpdate( Subject<VarPercent> &rBand, void *arg )
 {
+    (void)rBand; (void)arg;
     aout_instance_t *pAout = NULL;
 
     playlist_t *pPlaylist = getIntf()->p_sys->p_playlist;

diff --git a/modules/gui/skins2/x11/x11_display.cpp b/modules/gui/skins2/x11/x11_display.cpp
index dc42dd1316484d3564f405ef6b97764798236637..d259556d890dcf1b718001409ea45e8ab27ea93d 100644 (file)
--- a/modules/gui/skins2/x11/x11_display.cpp
+++ b/modules/gui/skins2/x11/x11_display.cpp
@@ -316,7 +316,7 @@ void X11Display::testEWMH()
 #define TEST_EWMH( name, value ) \
 {\
     name = XInternAtom( m_pDisplay, value, False );\
-    int i;\
+    unsigned int i;\
     for( i = 0; i < i_items; i++ )\
     {\
         if( p_args.p_atom[i] == name ) break;\
@@ -428,6 +428,7 @@ void X11Display::blendPixel32LSB( uint8_t *pPixel, uint8_t r, uint8_t g,
 void X11Display::putPixel8( uint8_t *pPixel, uint8_t r, uint8_t g,
                             uint8_t b, uint8_t a ) const
 {
+    (void)a;
     *pPixel = 255 - putPixel<uint8_t>(r,g,b);
 }
 
@@ -435,6 +436,7 @@ void X11Display::putPixel8( uint8_t *pPixel, uint8_t r, uint8_t g,
 void X11Display::putPixel16MSB( uint8_t *pPixel, uint8_t r, uint8_t g,
                                 uint8_t b, uint8_t a ) const
 {
+    (void)a;
     uint16_t value = putPixel<uint16_t>(r, g, b);
 
     pPixel[1] = value; value >>= 8;
@@ -445,6 +447,7 @@ void X11Display::putPixel16MSB( uint8_t *pPixel, uint8_t r, uint8_t g,
 void X11Display::putPixel16LSB( uint8_t *pPixel, uint8_t r, uint8_t g,
                                 uint8_t b, uint8_t a ) const
 {
+    (void)a;
     uint16_t value = putPixel<uint16_t>(r,g,b);
     pPixel[0] = value; value >>= 8;
     pPixel[1] = value;
@@ -454,6 +457,7 @@ void X11Display::putPixel16LSB( uint8_t *pPixel, uint8_t r, uint8_t g,
 void X11Display::putPixel32MSB( uint8_t *pPixel, uint8_t r, uint8_t g,
                                 uint8_t b, uint8_t a ) const
 {
+    (void)a;
     uint32_t value = putPixel<uint32_t>(r,g,b);
 
     pPixel[3] = value; value >>= 8;
@@ -466,6 +470,7 @@ void X11Display::putPixel32MSB( uint8_t *pPixel, uint8_t r, uint8_t g,
 void X11Display::putPixel32LSB( uint8_t *pPixel, uint8_t r, uint8_t g,
                                 uint8_t b, uint8_t a ) const
 {
+    (void)a;
     uint32_t value = putPixel<uint32_t>(r,g,b);
 
     pPixel[0] = value; value >>= 8;

diff --git a/modules/gui/skins2/x11/x11_dragdrop.cpp b/modules/gui/skins2/x11/x11_dragdrop.cpp
index e8111191475927b7b0df7fc55046f35d5123c7ef..a0062448caec706ab3a08bbde64c04481758bfca 100644 (file)
--- a/modules/gui/skins2/x11/x11_dragdrop.cpp
+++ b/modules/gui/skins2/x11/x11_dragdrop.cpp
@@ -136,6 +136,7 @@ void X11DragDrop::dndPosition( ldata_t data )
 
 void X11DragDrop::dndLeave( ldata_t data )
 {
+    (void)data;
 }
 
 

diff --git a/modules/gui/skins2/x11/x11_factory.hpp b/modules/gui/skins2/x11/x11_factory.hpp
index b53ae233717f3cc3e6e6698765d6c91571c4379f..e6d98c90b8cabba8732e1be3909a158c3e62e87b 100644 (file)
--- a/modules/gui/skins2/x11/x11_factory.hpp
+++ b/modules/gui/skins2/x11/x11_factory.hpp
@@ -134,7 +134,8 @@ public:
     virtual void getMousePos( int &rXPos, int &rYPos ) const;
 
     /// Change the cursor
-    virtual void changeCursor( CursorType_t type ) const { /*TODO*/ }
+    virtual void changeCursor( CursorType_t type ) const
+        { /*TODO*/ (void)type; }
 
     /// Delete a directory recursively
     virtual void rmDir( const string &rPath );

diff --git a/modules/gui/skins2/x11/x11_loop.cpp b/modules/gui/skins2/x11/x11_loop.cpp
index f3a362080b3ae5930a7d5ceed722ccf913e6ab14..0283042bdd9cbfe3a4e45b96fe0749b7a968e5da 100644 (file)
--- a/modules/gui/skins2/x11/x11_loop.cpp
+++ b/modules/gui/skins2/x11/x11_loop.cpp
@@ -176,7 +176,7 @@ void X11Loop::handleX11Event()
                 XInternAtom( XDISPLAY, "WM_DELETE_WINDOW", False);
 
             if( event.xclient.message_type == wm_protocols &&
-                event.xclient.data.l[0] == wm_delete )
+                (Atom)event.xclient.data.l[0] == wm_delete )
             {
                 msg_Dbg( getIntf(), "Received WM_DELETE_WINDOW message" );
                 libvlc_Quit( getIntf()->p_libvlc );

diff --git a/modules/gui/skins2/x11/x11_popup.cpp b/modules/gui/skins2/x11/x11_popup.cpp
index e073bbbb4c292e2a6997af5f2f2ed854bb20c08f..bdf91fbfa53af06566db854d2e229d715a834d40 100644 (file)
--- a/modules/gui/skins2/x11/x11_popup.cpp
+++ b/modules/gui/skins2/x11/x11_popup.cpp
@@ -30,6 +30,7 @@ X11Popup::X11Popup( intf_thread_t *pIntf, X11Display &rDisplay )
     : OSPopup( pIntf )
 {
     // TODO
+    (void)rDisplay;
 }
 
 
@@ -42,6 +43,7 @@ X11Popup::~X11Popup()
 void X11Popup::show( int xPos, int yPos )
 {
     // TODO
+    (void)xPos; (void)yPos;
 }
 
 
@@ -54,18 +56,21 @@ void X11Popup::hide()
 void X11Popup::addItem( const string &rLabel, int pos )
 {
     // TODO
+    (void)rLabel; (void)pos;
 }
 
 
 void X11Popup::addSeparator( int pos )
 {
     // TODO
+    (void)pos;
 }
 
 
 int X11Popup::getPosFromId( int id ) const
 {
     // TODO
+    (void)id;
     return 0;
 }