Commit 7924faca authored by Pierre d'Herbemont's avatar Pierre d'Herbemont

cdda: Avoid a macro and kill a warning.

parent d8dc5c85
...@@ -157,15 +157,15 @@ cddb_end: ; ...@@ -157,15 +157,15 @@ cddb_end: ;
} }
#endif /*HAVE_LIBCDDB*/ #endif /*HAVE_LIBCDDB*/
#define add_meta_val(VLC_META, VAL) \ static inline void add_meta_val(access_t *p_access, vlc_meta_type_t meta, const char * val)
if ( p_cdda->p_meta && VAL) { \ {
/*vlc_meta_Add( p_cdda->p_meta, VLC_META, VAL );*/ \ cdda_data_t *p_cdda = (cdda_data_t *)p_access->p_sys;
dbg_print( INPUT_DBG_META, "field %s: %s\n", \ if (p_cdda->p_meta && val)
input_MetaTypeToLocalizedString(VLC_META), VAL ); \ dbg_print( INPUT_DBG_META, "field %s: %s\n", input_MetaTypeToLocalizedString(meta), val);
} \ }
#define add_cddb_meta(FIELD, VLC_META) \ #define add_cddb_meta(FIELD, VLC_META) \
add_meta_val(VLC_META, cddb_disc_get_##FIELD(p_cdda->cddb.disc)); add_meta_val(p_access, VLC_META, cddb_disc_get_##FIELD(p_cdda->cddb.disc));
#define add_cddb_meta_fmt(FIELD, FORMAT_SPEC, VLC_META) \ #define add_cddb_meta_fmt(FIELD, FORMAT_SPEC, VLC_META) \
{ \ { \
...@@ -173,7 +173,7 @@ cddb_end: ; ...@@ -173,7 +173,7 @@ cddb_end: ;
snprintf( psz_buf, sizeof(psz_buf)-1, FORMAT_SPEC, \ snprintf( psz_buf, sizeof(psz_buf)-1, FORMAT_SPEC, \
cddb_disc_get_##FIELD(p_cdda->cddb.disc)); \ cddb_disc_get_##FIELD(p_cdda->cddb.disc)); \
psz_buf[sizeof(psz_buf)-1] = '\0'; \ psz_buf[sizeof(psz_buf)-1] = '\0'; \
add_meta_val(VLC_META, psz_buf); \ add_meta_val(p_access, VLC_META, psz_buf); \
} }
/* Adds a string-valued entry to the stream and media information if /* Adds a string-valued entry to the stream and media information if
...@@ -302,11 +302,11 @@ CDDAMetaInfo( access_t *p_access, track_t i_track ) ...@@ -302,11 +302,11 @@ CDDAMetaInfo( access_t *p_access, track_t i_track )
{ {
if( cddb_track_get_title(t) != NULL && ! p_cdda->b_nav_mode ) if( cddb_track_get_title(t) != NULL && ! p_cdda->b_nav_mode )
{ {
add_meta_val( vlc_meta_Title, cddb_track_get_title(t) ); add_meta_val( p_access, vlc_meta_Title, cddb_track_get_title(t) );
} }
if( cddb_track_get_artist(t) != NULL ) if( cddb_track_get_artist(t) != NULL )
{ {
add_meta_val( vlc_meta_Artist, cddb_track_get_artist(t) ); add_meta_val( p_access, vlc_meta_Artist, cddb_track_get_artist(t) );
} }
} }
} }
...@@ -452,14 +452,14 @@ CDDAMetaInfo( access_t *p_access, track_t i_track ) ...@@ -452,14 +452,14 @@ CDDAMetaInfo( access_t *p_access, track_t i_track )
{ {
char *psz_name = CDDAFormatTitle( p_access, i_track ) ; char *psz_name = CDDAFormatTitle( p_access, i_track ) ;
if ( !p_cdda->b_nav_mode ) { if ( !p_cdda->b_nav_mode ) {
add_meta_val( vlc_meta_Title, psz_name ); add_meta_val( p_access, vlc_meta_Title, psz_name );
} else } else
{ {
input_Control( p_cdda->p_input, INPUT_SET_NAME, psz_name ); input_Control( p_cdda->p_input, INPUT_SET_NAME, psz_name );
free(psz_name); free(psz_name);
} }
if (psz_meta_artist) if (psz_meta_artist)
add_meta_val( vlc_meta_Artist, psz_meta_artist ); add_meta_val( p_access, vlc_meta_Artist, psz_meta_artist );
} }
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment