/*****************************************************************************
* cdda.c : CD digital audio input module for vlc
*****************************************************************************
- * Copyright (C) 2000, 2003 the VideoLAN team
+ * Copyright (C) 2000, 2003-2006, 2008-2009 VLC authors and VideoLAN
* $Id$
*
* Authors: Laurent Aimar <fenrir@via.ecp.fr>
* Gildas Bazin <gbazin@netcourrier.com>
*
- * 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
- * the Free Software Foundation; either version 2 of the License, or
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU Lesser General Public License as published by
+ * the Free Software Foundation; either version 2.1 of the License, or
* (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU Lesser General Public License for more details.
*
- * 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., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with this program; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
*****************************************************************************/
/**
* Todo:
- * - Improve CDDB support (non-blocking, cache, ...)
+ * - Improve CDDB support (non-blocking, ...)
* - Fix tracknumber in MRL
*/
#ifdef HAVE_CONFIG_H
# include "config.h"
#endif
+#include <assert.h>
#include <vlc_common.h>
#include <vlc_plugin.h>
#include <vlc_input.h>
#include <vlc_access.h>
#include <vlc_meta.h>
-#include <vlc_charset.h>
+#include <vlc_charset.h> /* ToLocaleDup */
#include <vlc_codecs.h> /* For WAVEHEADER */
-#include "vcd/cdrom.h"
+#include "vcd/cdrom.h" /* For CDDA_DATA_SIZE */
#ifdef HAVE_LIBCDDB
-#include <cddb/cddb.h>
-#endif
-
-#ifdef HAVE_ERRNO_H
-#include <errno.h>
+ #include <cddb/cddb.h>
+ #include <errno.h>
#endif
/*****************************************************************************
static int Open ( vlc_object_t * );
static void Close( vlc_object_t * );
-#define CACHING_TEXT N_("Caching value in ms")
-#define CACHING_LONGTEXT N_( \
- "Default caching value for Audio CDs. This " \
- "value should be set in milliseconds." )
-
vlc_module_begin ()
- set_shortname( N_("Audio CD"))
+ set_shortname( N_("Audio CD") )
set_description( N_("Audio CD input") )
set_capability( "access", 10 )
set_category( CAT_INPUT )
set_callbacks( Open, Close )
add_usage_hint( N_("[cdda:][device][@[track]]") )
- add_integer( "cdda-caching", DEFAULT_PTS_DELAY / 1000, NULL, CACHING_TEXT,
- CACHING_LONGTEXT, true )
-
- add_integer( "cdda-track", 0 , NULL, NULL, NULL, true )
- change_internal ()
- add_integer( "cdda-first-sector", -1, NULL, NULL, NULL, true )
- change_internal ()
- add_integer( "cdda-last-sector", -1, NULL, NULL, NULL, true )
- change_internal ()
-
- add_string( "cddb-server", "freedb.freedb.org", NULL,
- N_( "CDDB Server" ), N_( "Address of the CDDB server to use." ),
- true )
- add_integer( "cddb-port", 8880, NULL,
- N_( "CDDB port" ), N_( "CDDB Server port to use." ),
- true )
- add_shortcut( "cdda" )
- add_shortcut( "cddasimple" )
+ add_integer( "cdda-track", 0 , NULL, NULL, true )
+ change_volatile ()
+ add_integer( "cdda-first-sector", -1, NULL, NULL, true )
+ change_volatile ()
+ add_integer( "cdda-last-sector", -1, NULL, NULL, true )
+ change_volatile ()
+
+#ifdef HAVE_LIBCDDB
+ add_string( "cddb-server", "freedb.videolan.org", N_( "CDDB Server" ),
+ N_( "Address of the CDDB server to use." ), true )
+ add_integer( "cddb-port", 80, N_( "CDDB port" ),
+ N_( "CDDB Server port to use." ), true )
+ change_integer_range( 1, 65535 )
+#endif
+
+ add_shortcut( "cdda", "cddasimple" )
vlc_module_end ()
struct access_sys_t
{
vcddev_t *vcddev; /* vcd device descriptor */
+ uint64_t size;
/* Current position */
int i_sector; /* Current Sector */
- int * p_sectors; /* Track sectors */
+ int *p_sectors; /* Track sectors */
/* Wave header for the output data */
WAVEHEADER waveheader;
- bool b_header;
+ bool b_header;
int i_track;
int i_first_sector;
int i_last_sector;
-
-#ifdef HAVE_LIBCDDB
- cddb_disc_t *p_disc;
-#endif
};
static block_t *Block( access_t * );
-static int Seek( access_t *, int64_t );
+static int Seek( access_t *, uint64_t );
static int Control( access_t *, int, va_list );
static int GetTracks( access_t *p_access, input_item_t *p_current );
#ifdef HAVE_LIBCDDB
-static void GetCDDBInfo( access_t *p_access, int i_titles, int *p_sectors );
+static cddb_disc_t *GetCDDBInfo( access_t *p_access, int i_titles, int *p_sectors );
#endif
/*****************************************************************************
static int Open( vlc_object_t *p_this )
{
access_t *p_access = (access_t*)p_this;
- access_sys_t *p_sys;
- vcddev_t *vcddev;
- char *psz_name;
- int i_ret;
+ vcddev_t *vcddev;
+ char *psz_name;
- if( !p_access->psz_path || !*p_access->psz_path )
+ if( !p_access->psz_filepath || !*p_access->psz_filepath )
{
/* Only when selected */
- if( !p_this->b_force ) return VLC_EGENERIC;
+ if( !p_access->psz_access || !*p_access->psz_access )
+ return VLC_EGENERIC;
- psz_name = var_CreateGetString( p_this, "cd-audio" );
- if( !psz_name || !*psz_name )
- {
- free( psz_name );
+ psz_name = var_InheritString( p_this, "cd-audio" );
+ if( !psz_name )
return VLC_EGENERIC;
- }
}
- else psz_name = ToLocaleDup( p_access->psz_path );
+ else psz_name = ToLocaleDup( p_access->psz_filepath );
-#ifdef WIN32
+#if defined( _WIN32 ) || defined( __OS2__ )
if( psz_name[0] && psz_name[1] == ':' &&
psz_name[2] == '\\' && psz_name[3] == '\0' ) psz_name[2] = '\0';
#endif
- /* Open CDDA */
- if( (vcddev = ioctl_Open( VLC_OBJECT(p_access), psz_name )) == NULL )
+ access_sys_t *p_sys = calloc( 1, sizeof (*p_sys) );
+ if( unlikely(p_sys == NULL) )
{
- msg_Warn( p_access, "could not open %s", psz_name );
free( psz_name );
- return VLC_EGENERIC;
+ return VLC_ENOMEM;
}
+ p_access->p_sys = p_sys;
+
+ /* Open CDDA */
+ vcddev = ioctl_Open( VLC_OBJECT(p_access), psz_name );
+ if( vcddev == NULL )
+ msg_Warn( p_access, "could not open %s", psz_name );
free( psz_name );
+ if( vcddev == NULL )
+ {
+ free( p_sys );
+ return VLC_EGENERIC;
+ }
- /* Set up p_access */
- STANDARD_BLOCK_ACCESS_INIT
p_sys->vcddev = vcddev;
- /* Do we play a single track ? */
- p_sys->i_track = var_CreateGetInteger( p_access, "cdda-track" ) - 1;
+ /* Do we play a single track ? */
+ p_sys->i_track = var_InheritInteger( p_access, "cdda-track" ) - 1;
- if( p_sys->i_track < 0 )
- {
+ if( p_sys->i_track < 0 )
+ {
/* We only do separate items if the whole disc is requested */
- input_thread_t *p_input = (input_thread_t*)vlc_object_find( p_access, VLC_OBJECT_INPUT, FIND_PARENT );
+ input_thread_t *p_input = access_GetParentInput( p_access );
- i_ret = -1;
+ int i_ret = -1;
if( p_input )
{
input_item_t *p_current = input_GetItem( p_input );
p_sys->waveheader.DataChunkID = VLC_FOURCC('d', 'a', 't', 'a');
p_sys->waveheader.DataLength = 0; /* we just don't know */
- p_sys->i_first_sector = var_CreateGetInteger( p_access,
- "cdda-first-sector" );
- p_sys->i_last_sector = var_CreateGetInteger( p_access,
- "cdda-last-sector" );
+ p_sys->i_first_sector = var_InheritInteger( p_access,
+ "cdda-first-sector" );
+ p_sys->i_last_sector = var_InheritInteger( p_access,
+ "cdda-last-sector" );
/* Tracknumber in MRL */
if( p_sys->i_first_sector < 0 || p_sys->i_last_sector < 0 )
{
}
p_sys->i_sector = p_sys->i_first_sector;
- p_access->info.i_size = (p_sys->i_last_sector - p_sys->i_first_sector)
+ p_sys->size = (p_sys->i_last_sector - p_sys->i_first_sector)
* (int64_t)CDDA_DATA_SIZE;
}
- /* PTS delay */
- var_Create( p_access, "cdda-caching", VLC_VAR_INTEGER|VLC_VAR_DOINHERIT );
-
+ /* Set up p_access */
+ access_InitFields( p_access );
+ ACCESS_SET_CALLBACKS( NULL, Block, Control, Seek );
return VLC_SUCCESS;
error:
if( !p_sys->b_header )
{
/* Return only the header */
- p_block = block_New( p_access, sizeof( WAVEHEADER ) );
+ p_block = block_Alloc( sizeof( WAVEHEADER ) );
memcpy( p_block->p_buffer, &p_sys->waveheader, sizeof(WAVEHEADER) );
p_sys->b_header = true;
return p_block;
i_blocks = p_sys->i_last_sector - p_sys->i_sector;
/* Do the actual reading */
- if( !( p_block = block_New( p_access, i_blocks * CDDA_DATA_SIZE ) ) )
+ if( !( p_block = block_Alloc( i_blocks * CDDA_DATA_SIZE ) ) )
{
msg_Err( p_access, "cannot get a new block of size: %i",
i_blocks * CDDA_DATA_SIZE );
/****************************************************************************
* Seek
****************************************************************************/
-static int Seek( access_t *p_access, int64_t i_pos )
+static int Seek( access_t *p_access, uint64_t i_pos )
{
access_sys_t *p_sys = p_access->p_sys;
/* Next sector to read */
p_sys->i_sector = p_sys->i_first_sector + i_pos / CDDA_DATA_SIZE;
+ assert( p_sys->i_sector >= 0 );
p_access->info.i_pos = i_pos;
return VLC_SUCCESS;
*****************************************************************************/
static int Control( access_t *p_access, int i_query, va_list args )
{
- bool *pb_bool;
- int64_t *pi_64;
-
switch( i_query )
{
case ACCESS_CAN_SEEK:
case ACCESS_CAN_FASTSEEK:
case ACCESS_CAN_PAUSE:
case ACCESS_CAN_CONTROL_PACE:
- pb_bool = (bool*)va_arg( args, bool* );
- *pb_bool = true;
+ *va_arg( args, bool* ) = true;
+ break;
+ case ACCESS_GET_SIZE:
+ *va_arg( args, uint64_t * ) = p_access->p_sys->size;
break;
-
case ACCESS_GET_PTS_DELAY:
- pi_64 = (int64_t*)va_arg( args, int64_t * );
- *pi_64 = var_GetInteger( p_access, "cdda-caching" ) * INT64_C(1000);
+ *va_arg( args, int64_t * ) =
+ INT64_C(1000) * var_InheritInteger( p_access, "disc-caching" );
break;
case ACCESS_SET_PAUSE_STATE:
break;
- case ACCESS_GET_TITLE_INFO:
- case ACCESS_SET_TITLE:
- case ACCESS_GET_META:
- case ACCESS_SET_SEEKPOINT:
- case ACCESS_SET_PRIVATE_ID_STATE:
- case ACCESS_GET_CONTENT_TYPE:
- return VLC_EGENERIC;
-
default:
- msg_Warn( p_access, "unimplemented query in control" );
return VLC_EGENERIC;
}
return VLC_SUCCESS;
return VLC_EGENERIC;;
}
+ /* */
input_item_SetName( p_current, "Audio CD" );
-#ifdef HAVE_LIBCDDB
- GetCDDBInfo( p_access, i_titles, p_sys->p_sectors );
const char *psz_album = NULL;
const char *psz_year = NULL;
const char *psz_genre = NULL;
- char psz_year_buffer[4+1];
+ const char *psz_artist = NULL;
+ const char *psz_description = NULL;
- if( p_sys->p_disc )
+/* Return true if the given string is not NULL and not empty */
+#define NONEMPTY( psz ) ( (psz) && *(psz) )
+/* If the given string is NULL or empty, fill it by the return value of 'code' */
+#define ON_EMPTY( psz, code ) do { if( !NONEMPTY( psz) ) { (psz) = code; } } while(0)
+
+ /* Retreive CDDB information */
+#ifdef HAVE_LIBCDDB
+ char psz_year_buffer[4+1];
+ msg_Dbg( p_access, "fetching infos with CDDB" );
+ cddb_disc_t *p_disc = GetCDDBInfo( p_access, i_titles, p_sys->p_sectors );
+ if( p_disc )
{
- psz_album = cddb_disc_get_title( p_sys->p_disc );
- if( psz_album && *psz_album )
- {
- input_item_SetName( p_current, psz_album );
- input_item_SetAlbum( p_current, psz_album );
- }
+ msg_Dbg( p_access, "Disc ID: %08x", cddb_disc_get_discid( p_disc ) );
+ psz_album = cddb_disc_get_title( p_disc );
+ psz_genre = cddb_disc_get_genre( p_disc );
- const unsigned i_year = cddb_disc_get_year( p_sys->p_disc );
+ /* */
+ const unsigned i_year = cddb_disc_get_year( p_disc );
if( i_year > 0 )
{
psz_year = psz_year_buffer;
snprintf( psz_year_buffer, sizeof(psz_year_buffer), "%u", i_year );
- input_item_SetDate( p_current, psz_year );
}
- psz_genre = cddb_disc_get_genre( p_sys->p_disc );
- if( psz_genre && *psz_genre )
- input_item_SetGenre( p_current, psz_genre );
-
/* Set artist only if unique */
- const char *psz_artist = NULL;
for( int i = 0; i < i_titles; i++ )
{
- cddb_track_t *t = cddb_disc_get_track( p_sys->p_disc, i );
+ cddb_track_t *t = cddb_disc_get_track( p_disc, i );
if( !t )
continue;
const char *psz_track_artist = cddb_track_get_artist( t );
}
psz_artist = psz_track_artist;
}
- if( psz_artist && *psz_artist )
- input_item_SetArtist( p_current, psz_artist );
-
- const mtime_t i_duration = (int64_t)( p_sys->p_sectors[i_titles] - p_sys->p_sectors[0] ) *
- CDDA_DATA_SIZE * 1000000 / 44100 / 2 / 2;
- input_item_SetDuration( p_current, i_duration );
}
+ else
+ msg_Dbg( p_access, "GetCDDBInfo failed" );
#endif
+ /* CD-Text */
+ vlc_meta_t **pp_cd_text;
+ int i_cd_text;
+
+ if( ioctl_GetCdText( VLC_OBJECT(p_access), p_sys->vcddev, &pp_cd_text, &i_cd_text ) )
+ {
+ msg_Dbg( p_access, "CD-TEXT information missing" );
+ i_cd_text = 0;
+ pp_cd_text = NULL;
+ }
+
+ /* Retrieve CD-TEXT information but prefer CDDB */
+ if( i_cd_text > 0 && pp_cd_text[0] )
+ {
+ const vlc_meta_t *p_disc = pp_cd_text[0];
+ ON_EMPTY( psz_album, vlc_meta_Get( p_disc, vlc_meta_Album ) );
+ ON_EMPTY( psz_genre, vlc_meta_Get( p_disc, vlc_meta_Genre ) );
+ ON_EMPTY( psz_artist, vlc_meta_Get( p_disc, vlc_meta_Artist ) );
+ ON_EMPTY( psz_description, vlc_meta_Get( p_disc, vlc_meta_Description ) );
+ }
+
+ if( NONEMPTY( psz_album ) )
+ {
+ input_item_SetName( p_current, psz_album );
+ input_item_SetAlbum( p_current, psz_album );
+ }
+
+ if( NONEMPTY( psz_genre ) )
+ input_item_SetGenre( p_current, psz_genre );
+
+ if( NONEMPTY( psz_artist ) )
+ input_item_SetArtist( p_current, psz_artist );
+
+ if( NONEMPTY( psz_year ) )
+ input_item_SetDate( p_current, psz_year );
+
+ if( NONEMPTY( psz_description ) )
+ input_item_SetDescription( p_current, psz_description );
+
+ const mtime_t i_duration = (int64_t)( p_sys->p_sectors[i_titles] - p_sys->p_sectors[0] ) *
+ CDDA_DATA_SIZE * 1000000 / 44100 / 2 / 2;
+ input_item_SetDuration( p_current, i_duration );
+
+ input_item_node_t *p_root = input_item_node_Create( p_current );
+
/* Build title table */
for( int i = 0; i < i_titles; i++ )
{
- input_item_t *p_input_item;
-
- char *psz_uri, *psz_opt, *psz_first, *psz_last;
- char *psz_name;
+ char *psz_uri, *psz_opt, *psz_name;
msg_Dbg( p_access, "track[%d] start=%d", i, p_sys->p_sectors[i] );
- /* */
- if( asprintf( &psz_uri, "cdda://%s", p_access->psz_path ) == -1 )
- psz_uri = NULL;
- if( asprintf( &psz_opt, "cdda-track=%i", i+1 ) == -1 )
- psz_opt = NULL;
- if( asprintf( &psz_first, "cdda-first-sector=%i",p_sys->p_sectors[i] ) == -1 )
- psz_first = NULL;
- if( asprintf( &psz_last, "cdda-last-sector=%i", p_sys->p_sectors[i+1] ) == -1 )
- psz_last = NULL;
+ if( asprintf( &psz_uri, "cdda://%s", p_access->psz_location ) == -1 )
+ continue;
/* Define a "default name" */
if( asprintf( &psz_name, _("Audio CD - Track %02i"), (i+1) ) == -1 )
- psz_name = NULL;
+ psz_name = psz_uri;
/* Create playlist items */
const mtime_t i_duration = (int64_t)( p_sys->p_sectors[i+1] - p_sys->p_sectors[i] ) *
CDDA_DATA_SIZE * 1000000 / 44100 / 2 / 2;
- p_input_item = input_item_NewWithType( VLC_OBJECT( p_access ),
- psz_uri, psz_name, 0, NULL, 0, i_duration,
- ITEM_TYPE_DISC );
- input_item_CopyOptions( p_current, p_input_item );
- input_item_AddOption( p_input_item, psz_first, VLC_INPUT_OPTION_TRUSTED );
- input_item_AddOption( p_input_item, psz_last, VLC_INPUT_OPTION_TRUSTED );
- input_item_AddOption( p_input_item, psz_opt, VLC_INPUT_OPTION_TRUSTED );
+
+ input_item_t *p_item = input_item_NewWithType( psz_uri, psz_name, 0,
+ NULL, 0, i_duration, ITEM_TYPE_DISC );
+ if( likely(psz_name != psz_uri) )
+ free( psz_name );
+ free( psz_uri );
+
+ if( unlikely(p_item == NULL) )
+ continue;
+
+ input_item_CopyOptions( p_current, p_item );
+
+ if( likely(asprintf( &psz_opt, "cdda-track=%i", i+1 ) != -1) )
+ {
+ input_item_AddOption( p_item, psz_opt, VLC_INPUT_OPTION_TRUSTED );
+ free( psz_opt );
+ }
+ if( likely(asprintf( &psz_opt, "cdda-first-sector=%i",
+ p_sys->p_sectors[i] ) != -1) )
+ {
+ input_item_AddOption( p_item, psz_opt, VLC_INPUT_OPTION_TRUSTED );
+ free( psz_opt );
+ }
+ if( likely(asprintf( &psz_opt, "cdda-last-sector=%i",
+ p_sys->p_sectors[i+1] ) != -1) )
+ {
+ input_item_AddOption( p_item, psz_opt, VLC_INPUT_OPTION_TRUSTED );
+ free( psz_opt );
+ }
+
+ const char *psz_track_title = NULL;
+ const char *psz_track_artist = NULL;
+ const char *psz_track_genre = NULL;
+ const char *psz_track_description = NULL;
#ifdef HAVE_LIBCDDB
- /* If we have CDDB info, change the name */
- if( p_sys->p_disc )
+ /* Retreive CDDB information */
+ if( p_disc )
{
- cddb_track_t *t = cddb_disc_get_track( p_sys->p_disc, i );
+ cddb_track_t *t = cddb_disc_get_track( p_disc, i );
if( t != NULL )
{
- const char *psz_title = cddb_track_get_title( t );
- const char *psz_artist = cddb_track_get_artist( t );
-
- if( psz_title )
- {
- input_item_SetName( p_input_item, psz_title );
- input_item_SetTitle( p_input_item, psz_title );
- }
- if( psz_artist )
- input_item_SetArtist( p_input_item, psz_artist );
+ psz_track_title = cddb_track_get_title( t );
+ psz_track_artist = cddb_track_get_artist( t );
}
+ }
+#endif
+
+ /* Retreive CD-TEXT information but prefer CDDB */
+ if( i+1 < i_cd_text && pp_cd_text[i+1] )
+ {
+ const vlc_meta_t *t = pp_cd_text[i+1];
- if( psz_album && *psz_album )
- input_item_SetAlbum( p_input_item, psz_album );
+ ON_EMPTY( psz_track_title, vlc_meta_Get( t, vlc_meta_Title ) );
+ ON_EMPTY( psz_track_artist, vlc_meta_Get( t, vlc_meta_Artist ) );
+ ON_EMPTY( psz_track_genre, vlc_meta_Get( t, vlc_meta_Genre ) );
+ ON_EMPTY( psz_track_description, vlc_meta_Get( t, vlc_meta_Description ) );
+ }
- if( psz_year )
- input_item_SetDate( p_input_item, psz_year );
+ /* */
+ ON_EMPTY( psz_track_artist, psz_artist );
+ ON_EMPTY( psz_track_genre, psz_genre );
+ ON_EMPTY( psz_track_description, psz_description );
- if( psz_genre && *psz_genre )
- input_item_SetGenre( p_input_item, psz_genre );
+ /* */
+ if( NONEMPTY( psz_track_title ) )
+ {
+ input_item_SetName( p_item, psz_track_title );
+ input_item_SetTitle( p_item, psz_track_title );
}
-#endif
+
+ if( NONEMPTY( psz_track_artist ) )
+ input_item_SetArtist( p_item, psz_track_artist );
+
+ if( NONEMPTY( psz_track_genre ) )
+ input_item_SetGenre( p_item, psz_track_genre );
+
+ if( NONEMPTY( psz_track_description ) )
+ input_item_SetDescription( p_item, psz_track_description );
+
+ if( NONEMPTY( psz_album ) )
+ input_item_SetAlbum( p_item, psz_album );
+
+ if( NONEMPTY( psz_year ) )
+ input_item_SetDate( p_item, psz_year );
+
char psz_num[3+1];
snprintf( psz_num, sizeof(psz_num), "%d", 1+i );
- input_item_SetTrackNum( p_input_item, psz_num );
+ input_item_SetTrackNum( p_item, psz_num );
- input_item_AddSubItem( p_current, p_input_item );
- vlc_gc_decref( p_input_item );
- free( psz_uri ); free( psz_opt ); free( psz_name );
- free( psz_first ); free( psz_last );
+ input_item_node_AppendItem( p_root, p_item );
+ vlc_gc_decref( p_item );
}
+#undef ON_EMPTY
+#undef NONEMPTY
+
+ input_item_node_PostAndDelete( p_root );
+
+ /* */
+ for( int i = 0; i < i_cd_text; i++ )
+ {
+ vlc_meta_t *p_meta = pp_cd_text[i];
+ if( !p_meta )
+ continue;
+ vlc_meta_Delete( p_meta );
+ }
+ free( pp_cd_text );
+
+#ifdef HAVE_LIBCDDB
+ if( p_disc )
+ cddb_disc_destroy( p_disc );
+#endif
return VLC_SUCCESS;
}
#ifdef HAVE_LIBCDDB
-static void GetCDDBInfo( access_t *p_access, int i_titles, int *p_sectors )
+static cddb_disc_t *GetCDDBInfo( access_t *p_access, int i_titles, int *p_sectors )
{
- int i, i_matches;
- int64_t i_length = 0, i_size = 0;
- cddb_conn_t *p_cddb = cddb_new();
+ if( var_InheritInteger( p_access, "album-art" ) != ALBUM_ART_ALL &&
+ ! var_InheritBool( p_access, "metadata-network-access" ) )
+ {
+ msg_Dbg( p_access, "Album art policy set to manual; no automatic fetching" );
+ return NULL;
+ }
+ /* */
+ cddb_conn_t *p_cddb = cddb_new();
if( !p_cddb )
{
msg_Warn( p_access, "unable to use CDDB" );
- goto cddb_destroy;
+ return NULL;
+ }
+
+ /* */
+
+ cddb_http_enable( p_cddb );
+
+ char *psz_tmp = var_InheritString( p_access, "cddb-server" );
+ if( psz_tmp )
+ {
+ cddb_set_server_name( p_cddb, psz_tmp );
+ free( psz_tmp );
}
- char* psz_tmp = config_GetPsz( p_access, "cddb-server" );
+ cddb_set_server_port( p_cddb, var_InheritInteger( p_access, "cddb-port" ) );
+
cddb_set_email_address( p_cddb, "vlc@videolan.org" );
- cddb_set_server_name( p_cddb, psz_tmp );
- cddb_set_server_port( p_cddb, config_GetInt( p_access, "cddb-port" ) );
- free( psz_tmp );
- /// \todo
- cddb_cache_disable( p_cddb );
+ cddb_set_http_path_query( p_cddb, "/~cddb/cddb.cgi" );
+ cddb_set_http_path_submit( p_cddb, "/~cddb/submit.cgi" );
-// cddb_cache_set_dir( p_cddb,
-// config_GetPsz( p_access,
-// MODULE_STRING "-cddb-cachedir") );
- cddb_set_timeout( p_cddb, 10 );
+ char *psz_cachedir;
+ char *psz_temp = config_GetUserDir( VLC_CACHE_DIR );
- /// \todo
- cddb_http_disable( p_cddb);
+ if( asprintf( &psz_cachedir, "%s" DIR_SEP "cddb", psz_temp ) > 0 ) {
+ cddb_cache_enable( p_cddb );
+ cddb_cache_set_dir( p_cddb, psz_cachedir );
+ free( psz_cachedir );
+ }
+ free( psz_temp );
- p_access->p_sys->p_disc = cddb_disc_new();
+ cddb_set_timeout( p_cddb, 10 );
- if(! p_access->p_sys->p_disc )
+ /* */
+ cddb_disc_t *p_disc = cddb_disc_new();
+ if( !p_disc )
{
msg_Err( p_access, "unable to create CDDB disc structure." );
- goto cddb_end;
+ goto error;
}
- for(i = 0; i < i_titles ; i++ )
+ int64_t i_length = 2000000; /* PreGap */
+ for( int i = 0; i < i_titles; i++ )
{
cddb_track_t *t = cddb_track_new();
- cddb_track_set_frame_offset(t, p_sectors[i] );
- cddb_disc_add_track( p_access->p_sys->p_disc, t );
- i_size = ( p_sectors[i+1] - p_sectors[i] ) *
- (int64_t)CDDA_DATA_SIZE;
+ cddb_track_set_frame_offset( t, p_sectors[i] + 150 ); /* Pregap offset */
+
+ cddb_disc_add_track( p_disc, t );
+ const int64_t i_size = ( p_sectors[i+1] - p_sectors[i] ) *
+ (int64_t)CDDA_DATA_SIZE;
i_length += INT64_C(1000000) * i_size / 44100 / 4 ;
+
+ msg_Dbg( p_access, "Track %i offset: %i", i, p_sectors[i] + 150 );
}
- cddb_disc_set_length( p_access->p_sys->p_disc, (int)(i_length/1000000) );
+ msg_Dbg( p_access, "Total length: %i", (int)(i_length/1000000) );
+ cddb_disc_set_length( p_disc, (int)(i_length/1000000) );
- if (!cddb_disc_calc_discid(p_access->p_sys->p_disc ))
+ if( !cddb_disc_calc_discid( p_disc ) )
{
msg_Err( p_access, "CDDB disc ID calculation failed" );
- goto cddb_destroy;
+ goto error;
}
- i_matches = cddb_query( p_cddb, p_access->p_sys->p_disc);
-
- if (i_matches > 0)
+ const int i_matches = cddb_query( p_cddb, p_disc );
+ if( i_matches < 0 )
{
- if (i_matches > 1)
- msg_Warn( p_access, "found %d matches in CDDB. Using first one.",
- i_matches);
- cddb_read( p_cddb, p_access->p_sys->p_disc );
+ msg_Warn( p_access, "CDDB error: %s", cddb_error_str(errno) );
+ goto error;
}
- else
- msg_Warn( p_access, "CDDB error: %s", cddb_error_str(errno));
+ else if( i_matches == 0 )
+ {
+ msg_Dbg( p_access, "Couldn't find any matches in CDDB." );
+ goto error;
+ }
+ else if( i_matches > 1 )
+ msg_Warn( p_access, "found %d matches in CDDB. Using first one.", i_matches );
+
+ cddb_read( p_cddb, p_disc );
-cddb_destroy:
cddb_destroy( p_cddb);
+ return p_disc;
-cddb_end: ;
+error:
+ if( p_disc )
+ cddb_disc_destroy( p_disc );
+ cddb_destroy( p_cddb );
+ return NULL;
}
#endif /*HAVE_LIBCDDB*/
+
projects / vlc / blobdiff