]> git.sesse.net Git - vlc/blobdiff - modules/gui/skins2/src/theme_loader.cpp
skins(Win32): replace GetWindowDC with GetDC
[vlc] / modules / gui / skins2 / src / theme_loader.cpp
index b48a45338eb6aa3d5faf9f10f9c56156f4e9c5b6..0d9cfd5b9ee7fede21f097417049b9b057588139 100644 (file)
@@ -5,7 +5,7 @@
  * $Id$
  *
  * Authors: Cyril Deguet     <asmax@via.ecp.fr>
- *          Olivier Teulière <ipkiss@via.ecp.fr>
+ *          Olivier Teulière <ipkiss@via.ecp.fr>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -19,7 +19,7 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111, USA.
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
  *****************************************************************************/
 
 #include "theme_loader.hpp"
@@ -50,7 +50,7 @@
 #if defined( HAVE_ZLIB_H )
 #   include <zlib.h>
 #   include <errno.h>
-int gzopen_frontend ( char *pathname, int oflags, int mode );
+int gzopen_frontend ( const char *pathname, int oflags, int mode );
 int gzclose_frontend( int );
 int gzread_frontend ( int, void *, size_t );
 int gzwrite_frontend( int, const void *, size_t );
@@ -73,11 +73,18 @@ int makedir( const char *newdir );
 
 bool ThemeLoader::load( const string &fileName )
 {
+    string path = getFilePath( fileName );
+
+    //Before all, let's see if the file is present
+    struct stat p_stat;
+    if( vlc_stat( path.c_str(), &p_stat ) )
+        return false;
+
     // First, we try to un-targz the file, and if it fails we hope it's a XML
     // file...
-    string path = getFilePath( fileName );
+
 #if defined( HAVE_ZLIB_H )
-    if( ! extract( fileName ) && ! parse( path, fileName ) )
+    if( ! extract( sToLocale( fileName ) ) && ! parse( path, fileName ) )
         return false;
 #else
     if( ! parse( path, fileName ) )
@@ -105,10 +112,7 @@ bool ThemeLoader::load( const string &fileName )
         // Show the windows
         pNewTheme->getWindowManager().showAll( true );
     }
-    if( skin_last ) free( skin_last );
-
-    // The new theme cannot embed a video output yet
-    VlcProc::instance( getIntf() )->dropVout();
+    free( skin_last );
 
     return true;
 }
@@ -163,8 +167,9 @@ bool ThemeLoader::extractZip( const string &zipFile, const string &rootDir )
     {
         if( !extractFileInZip( file, rootDir ) )
         {
-            msg_Warn( getIntf(), "Error while unzipping %s",
+            msg_Warn( getIntf(), "error while unzipping %s",
                       zipFile.c_str() );
+            unzClose( file );
             return false;
         }
 
@@ -173,12 +178,14 @@ bool ThemeLoader::extractZip( const string &zipFile, const string &rootDir )
             // Go the next file in the archive
             if( unzGoToNextFile( file ) !=UNZ_OK )
             {
-                msg_Warn( getIntf(), "Error while unzipping %s",
+                msg_Warn( getIntf(), "error while unzipping %s",
                           zipFile.c_str() );
+                unzClose( file );
                 return false;
             }
         }
     }
+    unzClose( file );
     return true;
 }
 
@@ -195,16 +202,27 @@ bool ThemeLoader::extractFileInZip( unzFile file, const string &rootDir )
         return false;
     }
 
+#ifdef WIN32
+
+    // Convert the file name to lower case, because some winamp skins
+    // use the wrong case...
+    for( size_t i=0; i< strlen( filenameInZip ); i++)
+    {
+        filenameInZip[i] = tolower( filenameInZip[i] );
+    }
+
+#endif
+
     // Allocate the buffer
     void *pBuffer = malloc( ZIP_BUFFER_SIZE );
     if( !pBuffer )
-    {
-        msg_Err( getIntf(), "Failed to allocate memory" );
         return false;
-    }
 
     // Get the path of the file
-    string fullPath = rootDir + "/" + filenameInZip;
+    OSFactory *pOsFactory = OSFactory::instance( getIntf() );
+    string fullPath = rootDir
+        + pOsFactory->getDirSeparator()
+        + fixDirSeparators( filenameInZip );
     string basePath = getFilePath( fullPath );
 
     // Extract the file if is not a directory
@@ -219,7 +237,7 @@ bool ThemeLoader::extractFileInZip( unzFile file, const string &rootDir )
         FILE *fout = fopen( fullPath.c_str(), "wb" );
         if( fout == NULL )
         {
-            msg_Err( getIntf(), "Error opening %s", fullPath.c_str() );
+            msg_Err( getIntf(), "error opening %s", fullPath.c_str() );
             free( pBuffer );
             return false;
         }
@@ -231,7 +249,7 @@ bool ThemeLoader::extractFileInZip( unzFile file, const string &rootDir )
             n = unzReadCurrentFile( file, pBuffer, ZIP_BUFFER_SIZE );
             if( n < 0 )
             {
-                msg_Err( getIntf(), "Error while reading zip file" );
+                msg_Err( getIntf(), "error while reading zip file" );
                 free( pBuffer );
                 return false;
             }
@@ -239,7 +257,7 @@ bool ThemeLoader::extractFileInZip( unzFile file, const string &rootDir )
             {
                 if( fwrite( pBuffer, n , 1, fout) != 1 )
                 {
-                    msg_Err( getIntf(), "Error while writing %s",
+                    msg_Err( getIntf(), "error while writing %s",
                              fullPath.c_str() );
                     free( pBuffer );
                     return false;
@@ -265,13 +283,16 @@ bool ThemeLoader::extract( const string &fileName )
 {
     bool result = true;
     char *tmpdir = tempnam( NULL, "vlt" );
-    string tempPath = tmpdir;
+    string tempPath = sFromLocale( tmpdir );
     free( tmpdir );
 
     // Extract the file in a temporary directory
     if( ! extractTarGz( fileName, tempPath ) &&
         ! extractZip( fileName, tempPath ) )
+    {
+        deleteTempFiles( tempPath );
         return false;
+    }
 
     string path;
     string xmlFile;
@@ -287,7 +308,7 @@ bool ThemeLoader::extract( const string &fileName )
         string mainBmp;
         if( findFile( tempPath, "main.bmp", mainBmp ) )
         {
-            msg_Dbg( getIntf(), "Try to load a winamp2 skin" );
+            msg_Dbg( getIntf(), "trying to load a winamp2 skin" );
             path = getFilePath( mainBmp );
 
             // Look for winamp2.xml in the resource path
@@ -306,13 +327,13 @@ bool ThemeLoader::extract( const string &fileName )
         // Parse the XML file
         if (! parse( path, xmlFile ) )
         {
-            msg_Err( getIntf(), "Error while parsing %s", xmlFile.c_str() );
+            msg_Err( getIntf(), "error while parsing %s", xmlFile.c_str() );
             result = false;
         }
     }
     else
     {
-        msg_Err( getIntf(), "No XML found in theme %s", fileName.c_str() );
+        msg_Err( getIntf(), "no XML found in theme %s", fileName.c_str() );
         result = false;
     }
 
@@ -332,18 +353,15 @@ void ThemeLoader::deleteTempFiles( const string &path )
 bool ThemeLoader::parse( const string &path, const string &xmlFile )
 {
     // File loaded
-    msg_Dbg( getIntf(), "Using skin file: %s", xmlFile.c_str() );
+    msg_Dbg( getIntf(), "using skin file: %s", xmlFile.c_str() );
 
     // Start the parser
     SkinParser parser( getIntf(), xmlFile, path );
     if( ! parser.parse() )
-    {
-        msg_Err( getIntf(), "Failed to parse %s", xmlFile.c_str() );
         return false;
-    }
 
     // Build and store the theme
-    Builder builder( getIntf(), parser.getData() );
+    Builder builder( getIntf(), parser.getData(), path );
     getIntf()->p_sys->p_theme = builder.build();
 
     return true;
@@ -372,6 +390,21 @@ string ThemeLoader::getFilePath( const string &rFullPath )
 }
 
 
+string ThemeLoader::fixDirSeparators( const string &rPath )
+{
+    OSFactory *pOsFactory = OSFactory::instance( getIntf() );
+    const string &sep = pOsFactory->getDirSeparator();
+    string::size_type p = rPath.find( "/", 0 );
+    string newPath = rPath;
+    while( p != string::npos )
+    {
+        newPath = newPath.replace( p, 1, sep );
+        p = newPath.find( "/", p + 1 );
+    }
+    return newPath;
+}
+
+
 bool ThemeLoader::findFile( const string &rootDir, const string &rFileName,
                             string &themeFilePath )
 {
@@ -379,34 +412,32 @@ bool ThemeLoader::findFile( const string &rootDir, const string &rFileName,
     const string &sep = OSFactory::instance( getIntf() )->getDirSeparator();
 
     DIR *pCurrDir;
-    struct dirent *pDirContent;
+    char *pszDirContent;
 
     // Open the dir
-    pCurrDir = opendir( rootDir.c_str() );
+    pCurrDir = vlc_opendir( rootDir.c_str() );
 
     if( pCurrDir == NULL )
     {
         // An error occurred
-        msg_Dbg( getIntf(), "Cannot open directory %s", rootDir.c_str() );
+        msg_Dbg( getIntf(), "cannot open directory %s", rootDir.c_str() );
         return false;
     }
 
-    // Get the first directory entry
-    pDirContent = (dirent*)readdir( pCurrDir );
-
     // While we still have entries in the directory
-    while( pDirContent != NULL )
+    while( ( pszDirContent = vlc_readdir( pCurrDir ) ) != NULL )
     {
-        string newURI = rootDir + sep + pDirContent->d_name;
+        string newURI = rootDir + sep + pszDirContent;
 
         // Skip . and ..
-        if( string( pDirContent->d_name ) != "." &&
-            string( pDirContent->d_name ) != ".." )
+        if( string( pszDirContent ) != "." &&
+            string( pszDirContent ) != ".." )
         {
 #if defined( S_ISDIR )
             struct stat stat_data;
-            stat( newURI.c_str(), &stat_data );
-            if( S_ISDIR(stat_data.st_mode) )
+
+            if( ( vlc_stat( newURI.c_str(), &stat_data ) == 0 )
+             && S_ISDIR(stat_data.st_mode) )
 #elif defined( DT_DIR )
             if( pDirContent->d_type & DT_DIR )
 #else
@@ -416,6 +447,7 @@ bool ThemeLoader::findFile( const string &rootDir, const string &rFileName,
                 // Can we find the file in this subdirectory?
                 if( findFile( newURI, rFileName, themeFilePath ) )
                 {
+                    free( pszDirContent );
                     closedir( pCurrDir );
                     return true;
                 }
@@ -423,16 +455,17 @@ bool ThemeLoader::findFile( const string &rootDir, const string &rFileName,
             else
             {
                 // Found the theme file?
-                if( rFileName == string( pDirContent->d_name ) )
+                if( rFileName == string( pszDirContent ) )
                 {
                     themeFilePath = newURI;
+                    free( pszDirContent );
                     closedir( pCurrDir );
                     return true;
                 }
             }
         }
 
-        pDirContent = (dirent*)readdir( pCurrDir );
+        free( pszDirContent );
     }
 
     closedir( pCurrDir );
@@ -544,34 +577,33 @@ int tar_extract_all( TAR *t, char *prefix )
 
             switch( buffer.header.typeflag )
             {
-                case DIRTYPE:
-                    makedir( fname );
-                    break;
-                case REGTYPE:
-                case AREGTYPE:
-                    remaining = getoct( buffer.header.size, 12 );
-                    if( remaining )
+            case DIRTYPE:
+                makedir( fname );
+                break;
+            case REGTYPE:
+            case AREGTYPE:
+                remaining = getoct( buffer.header.size, 12 );
+                if( !remaining ) outfile = NULL; else
+                {
+                    outfile = fopen( fname, "wb" );
+                    if( outfile == NULL )
                     {
-                        outfile = fopen( fname, "wb" );
-                        if( outfile == NULL )
+                        /* try creating directory */
+                        char *p = strrchr( fname, '/' );
+                        if( p != NULL )
                         {
-                            /* try creating directory */
-                            char *p = strrchr( fname, '/' );
-                            if( p != NULL )
+                            *p = '\0';
+                            makedir( fname );
+                            *p = '/';
+                            outfile = fopen( fname, "wb" );
+                            if( !outfile )
                             {
-                                *p = '\0';
-                                makedir( fname );
-                                *p = '/';
-                                outfile = fopen( fname, "wb" );
-                                if( !outfile )
-                                {
-                                    fprintf( stderr, "tar couldn't create %s\n",
-                                             fname );
-                                }
+                                fprintf( stderr, "tar couldn't create %s\n",
+                                         fname );
                             }
                         }
                     }
-                    else outfile = NULL;
+                }
 
                 /*
                  * could have no contents
@@ -592,6 +624,7 @@ int tar_extract_all( TAR *t, char *prefix )
                 {
                     fprintf( stderr, "error writing %s skipping...\n", fname );
                     fclose( outfile );
+                    outfile = NULL;
                     unlink( fname );
                 }
             }
@@ -698,23 +731,23 @@ int makedir( const char *newdir )
 static int currentGzFd = -1;
 static void * currentGzVp = NULL;
 
-int gzopen_frontend( char *pathname, int oflags, int mode )
+int gzopen_frontend( const char *pathname, int oflags, int mode )
 {
-    char *gzflags;
+    const char *gzflags;
     gzFile gzf;
 
     switch( oflags )
     {
-        case O_WRONLY:
-            gzflags = "wb";
-            break;
-        case O_RDONLY:
-            gzflags = "rb";
-            break;
-        case O_RDWR:
-        default:
-            errno = EINVAL;
-            return -1;
+    case O_WRONLY:
+        gzflags = "wb";
+        break;
+    case O_RDONLY:
+        gzflags = "rb";
+        break;
+    case O_RDWR:
+    default:
+        errno = EINVAL;
+        return -1;
     }
 
     gzf = gzopen( pathname, gzflags );