]> git.sesse.net Git - vlc/blobdiff - src/input/meta.c
input_CurrentMetaFlags() takes a const parameter
[vlc] / src / input / meta.c
index 280b2936e1e3d449198e44cbd2754378f37c1753..f1a05ebfd67689f337f4d749825061ffb74b0cbd 100644 (file)
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
  *****************************************************************************/
 
+#ifdef HAVE_CONFIG_H
+# include "config.h"
+#endif
+
 #include <vlc/vlc.h>
 #include <vlc_input.h>
 #include <vlc_stream.h>
@@ -31,6 +35,8 @@
 #include <vlc_strings.h>
 #include "../playlist/playlist_internal.h"
 #include <errno.h>
+#include <limits.h>                                             /* PATH_MAX */
+#include <assert.h>
 
 #ifdef HAVE_SYS_STAT_H
 #   include <sys/stat.h>
@@ -69,7 +75,7 @@ static int __input_FindArtInCache( vlc_object_t *, input_item_t *p_item );
 vlc_bool_t input_MetaSatisfied( playlist_t *p_playlist, input_item_t *p_item,
                                 uint32_t *pi_mandatory, uint32_t *pi_optional )
 {
-    (void)p_playlist;
+    VLC_UNUSED(p_playlist);
     *pi_mandatory = VLC_META_ENGINE_TITLE | VLC_META_ENGINE_ARTIST;
 
     uint32_t i_meta = input_CurrentMetaFlags( p_item->p_meta );
@@ -110,7 +116,7 @@ int input_MetaFetch( playlist_t *p_playlist, input_item_t *p_item )
 }
 
 /* Return codes:
- *   0 : Art is in cache
+ *   0 : Art is in cache or is a local file
  *   1 : Art found, need to download
  *  -X : Error/not found
  */
@@ -120,16 +126,13 @@ int input_ArtFind( playlist_t *p_playlist, input_item_t *p_item )
     module_t *p_module;
     char *psz_title, *psz_artist, *psz_album;
 
-    if( !p_item->p_meta )
-        return VLC_EGENERIC;
-
     psz_artist = input_item_GetArtist( p_item );
     psz_album = input_item_GetAlbum( p_item );
     psz_title = input_item_GetTitle( p_item );
     if(!psz_title)
         psz_title = input_item_GetName( p_item );
 
-    if(  !psz_title && !psz_artist && !psz_album )
+    if( !psz_title && !psz_artist && !psz_album )
         return VLC_EGENERIC;
 
     free( psz_title );
@@ -148,8 +151,10 @@ int input_ArtFind( playlist_t *p_playlist, input_item_t *p_item )
                 free( psz_album );
                 if( album.b_found )
                 {
-                    /* Actually get URL from cache */
-                    input_FindArtInCache( p_playlist, p_item );
+                    if( !strncmp( album.psz_arturl, "file://", 7 ) )
+                        input_item_SetArtURL( p_item, album.psz_arturl );
+                    else /* Actually get URL from cache */
+                        input_FindArtInCache( p_playlist, p_item );
                     return 0;
                 }
                 else
@@ -162,14 +167,23 @@ int input_ArtFind( playlist_t *p_playlist, input_item_t *p_item )
     free( psz_artist );
     free( psz_album );
 
-    char *psz_arturl = input_item_GetArtURL( p_item );
     input_FindArtInCache( p_playlist, p_item );
-    if( !EMPTY_STR( psz_arturl ) )
+
+    char *psz_arturl = input_item_GetArtURL( p_item );
+    if( psz_arturl )
     {
+        /* We already have an URL */
+        if( !strncmp( psz_arturl, "file://", strlen( "file://" ) ) )
+        {
+            free( psz_arturl );
+            return 0; /* Art is in cache, no need to go further */
+        }
+
         free( psz_arturl );
-        return 0;
+        
+        /* Art need to be put in cache */
+        return 1;
     }
-    free( psz_arturl );
 
     PL_LOCK;
     p_playlist->p_private = p_item;
@@ -204,6 +218,7 @@ int input_ArtFind( playlist_t *p_playlist, input_item_t *p_item )
         playlist_album_t a;
         a.psz_artist = psz_artist;
         a.psz_album = psz_album;
+        a.psz_arturl = input_item_GetArtURL( p_item );
         a.b_found = (i_ret == VLC_EGENERIC ? VLC_FALSE : VLC_TRUE );
         ARRAY_APPEND( p_playlist->p_fetcher->albums, a );
     }
@@ -221,24 +236,24 @@ int input_ArtFind( playlist_t *p_playlist, input_item_t *p_item )
     return i_ret;
 }
 
-#ifndef MAX_PATH
-#   define MAX_PATH 250
-#endif
 static void ArtCacheCreateDir( const char *psz_dir )
 {
-    char newdir[MAX_PATH+1];
+    char newdir[strlen( psz_dir ) + 1];
     strcpy( newdir, psz_dir );
     char * psz_newdir = newdir;
     char * psz = psz_newdir;
 
-    while( *psz_newdir )
+    while( *psz )
     {
-        while( *psz && *psz != '/') psz++;
+        while( *psz && *psz != DIR_SEP_CHAR) psz++;
         if( !*psz ) break;
-        *psz = 0;        
-        if( !EMPTY_STR( psz_newdir ) ) utf8_mkdir( psz_newdir );
-        psz_newdir = psz+1;
+        *psz = 0;
+        if( !EMPTY_STR( psz_newdir ) )
+            utf8_mkdir( psz_newdir, 0700 );
+        *psz = DIR_SEP_CHAR;
+        psz++;
     }
+    utf8_mkdir( psz_dir, 0700 );
 }
 
 static char * ArtCacheGetSanitizedFileName( const char *psz )
@@ -253,7 +268,7 @@ static char * ArtCacheGetSanitizedFileName( const char *psz )
      */
     for( i = 0; dup[i] != '\0'; i++ )
     {
-        if( dup[i] == '/' )
+        if( dup[i] == DIR_SEP_CHAR )
             dup[i] = ' ';
     }
     return dup;
@@ -270,9 +285,9 @@ static void __ArtCacheGetDirPath( vlc_object_t *p_obj,
         char * psz_album_sanitized = ArtCacheGetSanitizedFileName( psz_album );
         char * psz_artist_sanitized = ArtCacheGetSanitizedFileName( psz_artist );
 
-        snprintf( psz_dir, MAX_PATH, "%s" DIR_SEP CONFIG_DIR DIR_SEP
+        snprintf( psz_dir, PATH_MAX, "%s" DIR_SEP
                   "art" DIR_SEP "artistalbum" DIR_SEP "%s" DIR_SEP "%s",
-                      p_obj->p_libvlc->psz_homedir,
+                      p_obj->p_libvlc->psz_cachedir,
                       psz_artist_sanitized, psz_album_sanitized );
         free( psz_album_sanitized );
         free( psz_artist_sanitized );
@@ -280,9 +295,9 @@ static void __ArtCacheGetDirPath( vlc_object_t *p_obj,
     else
     {
         char * psz_title_sanitized = ArtCacheGetSanitizedFileName( psz_title );
-        snprintf( psz_dir, MAX_PATH, "%s" DIR_SEP CONFIG_DIR DIR_SEP
+        snprintf( psz_dir, PATH_MAX, "%s" DIR_SEP
                   "art" DIR_SEP "title" DIR_SEP "%s",
-                  p_obj->p_libvlc->psz_homedir,
+                  p_obj->p_libvlc->psz_cachedir,
                   psz_title_sanitized );
         free( psz_title_sanitized );
     }
@@ -297,7 +312,7 @@ static void __ArtCacheGetFilePath( vlc_object_t *p_obj,
                                    const char *psz_artist, const char *psz_album,
                                    const char *psz_extension )
 {
-    char psz_dir[MAX_PATH+1];
+    char psz_dir[PATH_MAX+1];
     char * psz_ext;
     ArtCacheGetDirPath( p_obj, psz_dir, psz_title, psz_artist, psz_album );
 
@@ -308,7 +323,7 @@ static void __ArtCacheGetFilePath( vlc_object_t *p_obj,
     }
     else psz_ext = strdup( "" );
 
-    snprintf( psz_filename, MAX_PATH, "file://%s" DIR_SEP "art%s",
+    snprintf( psz_filename, PATH_MAX, "file://%s" DIR_SEP "art%s",
               psz_dir, psz_ext );
 
     free( psz_ext );
@@ -319,13 +334,11 @@ static int __input_FindArtInCache( vlc_object_t *p_obj, input_item_t *p_item )
     char *psz_artist;
     char *psz_album;
     char *psz_title;
-    char psz_dirname[MAX_PATH+1];
-    char psz_filename[MAX_PATH+1];
-    struct dirent * p_de;
+    char psz_dirpath[PATH_MAX+1];
+    char psz_filepath[PATH_MAX+1];
+    char * psz_filename;
     DIR * p_dir;
 
-    if( !p_item->p_meta ) return VLC_EGENERIC;
-
     psz_artist = input_item_GetArtist( p_item );
     psz_album = input_item_GetAlbum( p_item );
     psz_title = input_item_GetTitle( p_item );
@@ -339,7 +352,7 @@ static int __input_FindArtInCache( vlc_object_t *p_obj, input_item_t *p_item )
         return VLC_EGENERIC;
     }
 
-    ArtCacheGetDirPath( p_obj, psz_dirname, psz_title,
+    ArtCacheGetDirPath( p_obj, psz_dirpath, psz_title,
                            psz_artist, psz_album );
 
     free( psz_artist );
@@ -347,22 +360,25 @@ static int __input_FindArtInCache( vlc_object_t *p_obj, input_item_t *p_item )
     free( psz_title );
 
     /* Check if file exists */
-    p_dir = utf8_opendir( psz_dirname );
+    p_dir = utf8_opendir( psz_dirpath );
     if( !p_dir )
         return VLC_EGENERIC;
 
-    while( (p_de = utf8_readdir( psz_dirname )) )
+    while( (psz_filename = utf8_readdir( p_dir )) )
     {
-        if( strncmp( p_de->d_name, "art", 3 ) )
+        if( !strncmp( psz_filename, "art", 3 ) )
         {
-            snprintf( psz_filename, MAX_PATH, "%s" DIR_SEP "%s",
-                            psz_dirname, p_de->d_name );
-            input_item_SetArtURL( p_item, psz_filename );
+            snprintf( psz_filepath, PATH_MAX, "file://%s" DIR_SEP "%s",
+                      psz_dirpath, psz_filename );
+            input_item_SetArtURL( p_item, psz_filepath );
+            free( psz_filename );
             closedir( p_dir );
             return VLC_SUCCESS;
         }
+        free( psz_filename );
     }
 
+    /* Not found */
     closedir( p_dir );
     return VLC_EGENERIC;
 }
@@ -375,7 +391,7 @@ int input_DownloadAndCacheArt( playlist_t *p_playlist, input_item_t *p_item )
 {
     int i_status = VLC_EGENERIC;
     stream_t *p_stream;
-    char psz_filename[MAX_PATH+1];
+    char psz_filename[PATH_MAX+1];
     char *psz_artist = NULL;
     char *psz_album = NULL;
     char *psz_title = NULL;
@@ -399,7 +415,13 @@ int input_DownloadAndCacheArt( playlist_t *p_playlist, input_item_t *p_item )
     psz_arturl = input_item_GetArtURL( p_item );
     assert( !EMPTY_STR( psz_arturl ) );
 
-    if( !strncmp( psz_arturl , "APIC", 4 ) )
+    if( !strncmp( psz_arturl , "file://", 7 ) )
+    {
+        msg_Dbg( p_playlist, "Album art is local file, no need to cache" );
+        free( psz_arturl );
+        return VLC_SUCCESS;
+    }
+    else if( !strncmp( psz_arturl , "APIC", 4 ) )
     {
         msg_Warn( p_playlist, "APIC fetch not supported yet" );
         free( psz_arturl );
@@ -407,7 +429,7 @@ int input_DownloadAndCacheArt( playlist_t *p_playlist, input_item_t *p_item )
     }
 
     psz_type = strrchr( psz_arturl, '.' );
-    if( strlen( psz_type ) > 5 ) 
+    if( psz_type && strlen( psz_type ) > 5 )
         psz_type = NULL; /* remove extension if it's > to 4 characters */
 
     /* Warning: psz_title, psz_artist, psz_album may change in ArtCache*() */
@@ -429,7 +451,8 @@ int input_DownloadAndCacheArt( playlist_t *p_playlist, input_item_t *p_item )
         long int l_read;
         FILE *p_file = utf8_fopen( psz_filename+7, "w" );
         if( p_file == NULL ) {
-            msg_Err( p_playlist, "Unable write album art in %s" );
+            msg_Err( p_playlist, "Unable write album art in %s",
+                     psz_filename + 7 );
             free( psz_arturl );
             return VLC_EGENERIC;
         }
@@ -447,7 +470,10 @@ int input_DownloadAndCacheArt( playlist_t *p_playlist, input_item_t *p_item )
         stream_Delete( p_stream );
 
         if( err )
-            msg_Err( p_playlist, "%s: %s", psz_filename, strerror( err ) );
+        {
+            errno = err;
+            msg_Err( p_playlist, "%s: %m", psz_filename );
+        }
         else
             msg_Dbg( p_playlist, "album art saved to %s\n", psz_filename );
 
@@ -461,12 +487,12 @@ int input_DownloadAndCacheArt( playlist_t *p_playlist, input_item_t *p_item )
 void input_ExtractAttachmentAndCacheArt( input_thread_t *p_input )
 {
     input_item_t *p_item = p_input->p->input.p_item;
-    char *psz_arturl;
-    char *psz_artist = NULL;
-    char *psz_album = NULL;
-    char *psz_title = NULL;
+    const char *psz_arturl;
+    const char *psz_artist = NULL;
+    const char *psz_album = NULL;
+    const char *psz_title = NULL;
     char *psz_type = NULL;
-    char psz_filename[MAX_PATH+1];
+    char psz_filename[PATH_MAX+1];
     FILE *f;
     input_attachment_t *p_attachment;
     struct stat s;
@@ -475,14 +501,13 @@ void input_ExtractAttachmentAndCacheArt( input_thread_t *p_input )
     /* TODO-fenrir merge input_ArtFind with download and make it set the flags FETCH
      * and then set it here to to be faster */
 
-    psz_arturl = input_item_GetArtURL( p_item );
+    psz_arturl = vlc_meta_Get( p_item->p_meta, vlc_meta_ArtworkURL );
+
     if( !psz_arturl || strncmp( psz_arturl, "attachment://", strlen("attachment://") ) )
     {
-        free( psz_arturl );
         msg_Err( p_input, "internal input error with input_ExtractAttachmentAndCacheArt" );
         return;
     }
-    input_item_SetArtURL( p_item, NULL );
 
     if( input_item_IsArtFetched( p_item ) )
     {
@@ -490,7 +515,6 @@ void input_ExtractAttachmentAndCacheArt( input_thread_t *p_input )
          * condition */
         msg_Warn( p_input, "internal input error with input_ExtractAttachmentAndCacheArt" );
         input_FindArtInCache( p_input, p_item );
-        free( psz_arturl );
         return;
     }
 
@@ -507,48 +531,51 @@ void input_ExtractAttachmentAndCacheArt( input_thread_t *p_input )
     if( !p_attachment || p_attachment->i_data <= 0 )
     {
         msg_Warn( p_input, "internal input error with input_ExtractAttachmentAndCacheArt" );
-        goto end;
+        return;
     }
 
-    psz_artist = input_item_GetArtist( p_item );
-    psz_album = input_item_GetAlbum( p_item );
-    psz_title = input_item_GetTitle( p_item );
+    psz_artist = vlc_meta_Get( p_item->p_meta, vlc_meta_Artist );
+    psz_album = vlc_meta_Get( p_item->p_meta, vlc_meta_Album );
+    psz_title = vlc_meta_Get( p_item->p_meta, vlc_meta_Title );
+    if( !strcmp( p_attachment->psz_mime, "image/jpeg" ) )
+        psz_type = strdup( ".jpg" );
+    else if( !strcmp( p_attachment->psz_mime, "image/png" ) )
+        psz_type = strdup( ".png" );
+
     if( !psz_title )
-        psz_title = input_item_GetName( p_item );
+        psz_title = p_item->psz_name;
 
     if( (!psz_artist || !psz_album ) && !psz_title )
-        goto end;
-
-    /* */
-    psz_type = strrchr( psz_arturl, '.' );
+        return;
 
     ArtCacheGetDirPath( p_input, psz_filename, psz_title, psz_artist, psz_album );
     ArtCacheCreateDir( psz_filename );
     ArtCacheGetFilePath( p_input, psz_filename, psz_title, psz_artist, psz_album, psz_type );
+    free( psz_type );
 
     /* Check if we already dumped it */
     if( !utf8_stat( psz_filename+7, &s ) )
-        goto end;
+    {
+        vlc_meta_Set( p_item->p_meta, vlc_meta_ArtworkURL, psz_filename );
+        return;
+    }
 
     f = utf8_fopen( psz_filename+7, "w" );
     if( f )
     {
         if( fwrite( p_attachment->p_data, p_attachment->i_data, 1, f ) != 1 )
-            msg_Err( p_input, "%s: %s", psz_filename, strerror( errno ) );
+            msg_Err( p_input, "%s: %m", psz_filename );
         else
+        {
             msg_Dbg( p_input, "album art saved to %s\n", psz_filename );
+            vlc_meta_Set( p_item->p_meta, vlc_meta_ArtworkURL, psz_filename );
+        }
         fclose( f );
     }
-
-end:
-    if( psz_artist ) free( psz_artist );
-    if( psz_album ) free( psz_album );
-    if( psz_title ) free( psz_title );
-    if( psz_arturl ) free( psz_arturl );
 }
 
 
-uint32_t input_CurrentMetaFlags( vlc_meta_t *p_meta )
+uint32_t input_CurrentMetaFlags( const vlc_meta_t *p_meta )
 {
     uint32_t i_meta = 0;