-
/*******************************************************************************
* xspf.c : XSPF playlist import functions
*******************************************************************************
{
input_item_t **pp_tracklist;
int i_tracklist_entries;
- int i_identifier;
+ int i_track_id;
char * psz_base;
};
void Close_xspf( vlc_object_t *p_this )
{
demux_t *p_demux = (demux_t *)p_this;
- int i;
- for(i = 0; i < p_demux->p_sys->i_tracklist_entries; i++)
+ demux_sys_t *p_sys = p_demux->p_sys;
+ for( int i = 0; i < p_sys->i_tracklist_entries; i++ )
{
- if(p_demux->p_sys->pp_tracklist[i])
- vlc_gc_decref( p_demux->p_sys->pp_tracklist[i] );
+ if( p_sys->pp_tracklist[i] )
+ vlc_gc_decref( p_sys->pp_tracklist[i] );
}
- FREENULL( p_demux->p_sys->pp_tracklist );
- FREENULL( p_demux->p_sys->psz_base );
- free( p_demux->p_sys );
+ free( p_sys->pp_tracklist );
+ free( p_sys->psz_base );
+ free( p_sys );
}
/**
*/
int Demux( demux_t *p_demux )
{
- int i_ret = 1;
- xml_t *p_xml = NULL;
+ int i_ret = -1;
xml_reader_t *p_xml_reader = NULL;
char *psz_name = NULL;
- INIT_PLAYLIST_STUFF;
+ input_item_t *p_current_input = GetCurrentItem(p_demux);
p_demux->p_sys->pp_tracklist = NULL;
p_demux->p_sys->i_tracklist_entries = 0;
- p_demux->p_sys->i_identifier = 0;
+ p_demux->p_sys->i_track_id = -1;
p_demux->p_sys->psz_base = NULL;
/* create new xml parser from stream */
- p_xml = xml_Create( p_demux );
- if( !p_xml )
- i_ret = -1;
- else
- {
- p_xml_reader = xml_ReaderCreate( p_xml, p_demux->s );
- if( !p_xml_reader )
- i_ret = -1;
- }
+ p_xml_reader = xml_ReaderCreate( p_demux, p_demux->s );
+ if( !p_xml_reader )
+ goto end;
/* locating the root node */
- if( i_ret == 1 )
+ do
{
- do
+ if( xml_ReaderRead( p_xml_reader ) != 1 )
{
- if( xml_ReaderRead( p_xml_reader ) != 1 )
- {
- msg_Err( p_demux, "can't read xml stream" );
- i_ret = -1;
- }
- } while( i_ret == VLC_SUCCESS &&
- xml_ReaderNodeType( p_xml_reader ) != XML_READER_STARTELEM );
- }
+ msg_Err( p_demux, "can't read xml stream" );
+ goto end;
+ }
+ } while( xml_ReaderNodeType( p_xml_reader ) != XML_READER_STARTELEM );
+
/* checking root node name */
- if( i_ret == 1 )
+ psz_name = xml_ReaderName( p_xml_reader );
+ if( !psz_name || strcmp( psz_name, "playlist" ) )
{
- psz_name = xml_ReaderName( p_xml_reader );
- if( !psz_name || strcmp( psz_name, "playlist" ) )
- {
- msg_Err( p_demux, "invalid root node name: %s", psz_name );
- i_ret = -1;
- }
- FREE_NAME();
+ msg_Err( p_demux, "invalid root node name: %s", psz_name );
+ free( psz_name );
+ goto end;
}
+ free( psz_name );
+
+ input_item_node_t *p_subitems =
+ input_item_node_Create( p_current_input );
- if( i_ret == 1 )
- i_ret = parse_playlist_node( p_demux, p_current_input,
- p_xml_reader, "playlist" ) ? 0 : -1;
+ i_ret = parse_playlist_node( p_demux, p_subitems,
+ p_xml_reader, "playlist" ) ? 0 : -1;
- int i;
- for( i = 0 ; i < p_demux->p_sys->i_tracklist_entries ; i++ )
+ for( int i = 0 ; i < p_demux->p_sys->i_tracklist_entries ; i++ )
{
input_item_t *p_new_input = p_demux->p_sys->pp_tracklist[i];
if( p_new_input )
{
- input_ItemAddSubItem( p_current_input, p_new_input );
+ input_item_node_AppendItem( p_subitems, p_new_input );
}
}
- HANDLE_PLAY_AND_RELEASE;
+ input_item_node_PostAndDelete( p_subitems );
+
+end:
+ vlc_gc_decref(p_current_input);
if( p_xml_reader )
- xml_ReaderDelete( p_xml, p_xml_reader );
- if( p_xml )
- xml_Delete( p_xml );
+ xml_ReaderDelete( p_xml_reader );
return i_ret; /* Needed for correct operation of go back */
}
*/
static bool parse_playlist_node COMPLEX_INTERFACE
{
- char *psz_name=NULL;
- char *psz_value=NULL;
+ input_item_t *p_input_item = p_input_node->p_item;
+ char *psz_name = NULL;
+ char *psz_value = NULL;
bool b_version_found = false;
int i_node;
- xml_elem_hnd_t *p_handler=NULL;
+ bool b_ret = false;
+ xml_elem_hnd_t *p_handler = NULL;
xml_elem_hnd_t pl_elements[] =
{ {"title", SIMPLE_CONTENT, {.smpl = set_item_info} },
if( !psz_name || !psz_value )
{
msg_Err( p_demux, "invalid xml stream @ <playlist>" );
- FREE_ATT();
- return false;
+ goto end;
}
/* attribute: version */
if( !strcmp( psz_name, "version" ) )
msg_Warn( p_demux, "unsupported XSPF version" );
}
/* attribute: xmlns */
- else if( !strcmp( psz_name, "xmlns" ) )
+ else if( !strcmp( psz_name, "xmlns" ) || !strcmp( psz_name, "xmlns:vlc" ) )
;
else if( !strcmp( psz_name, "xml:base" ) )
{
- p_demux->p_sys->psz_base = decode_URI_duplicate( psz_value );
+ p_demux->p_sys->psz_base = strdup( psz_value );
}
/* unknown attribute */
else
msg_Warn( p_demux, "invalid <playlist> attribute:\"%s\"", psz_name);
- FREE_ATT();
+ free( psz_name );
+ free( psz_value );
}
/* attribute version is mandatory !!! */
if( !b_version_found )
msg_Warn( p_demux, "<playlist> requires \"version\" attribute" );
/* parse the child elements - we only take care of <trackList> */
+ psz_name = NULL;
+ psz_value = NULL;
while( xml_ReaderRead( p_xml_reader ) == 1 )
{
i_node = xml_ReaderNodeType( p_xml_reader );
if( !psz_name || !*psz_name )
{
msg_Err( p_demux, "invalid xml stream" );
- FREE_ATT();
- return false;
+ goto end;
}
/* choose handler */
for( p_handler = pl_elements;
if( !p_handler->name )
{
msg_Err( p_demux, "unexpected element <%s>", psz_name );
- FREE_ATT();
- return false;
+ goto end;
}
FREE_NAME();
/* complex content is parsed in a separate function */
if( p_handler->type == COMPLEX_CONTENT )
{
+ FREE_VALUE();
if( p_handler->pf_handler.cmplx( p_demux,
- p_input_item,
+ p_input_node,
p_xml_reader,
p_handler->name ) )
{
p_handler = NULL;
- FREE_ATT();
}
else
{
- FREE_ATT();
return false;
}
}
case XML_READER_TEXT:
/* simple element content */
- FREE_ATT();
+ free( psz_value );
psz_value = xml_ReaderValue( p_xml_reader );
if( !psz_value )
{
msg_Err( p_demux, "invalid xml stream" );
- FREE_ATT();
- return false;
+ goto end;
}
break;
if( !psz_name )
{
msg_Err( p_demux, "invalid xml stream" );
- FREE_ATT();
- return false;
+ goto end;
}
/* leave if the current parent node <playlist> is terminated */
if( !strcmp( psz_name, psz_element ) )
{
- FREE_ATT();
- return true;
+ b_ret = true;
+ goto end;
}
/* there MUST have been a start tag for that element name */
if( !p_handler || !p_handler->name
{
msg_Err( p_demux, "there's no open element left for <%s>",
psz_name );
- FREE_ATT();
- return false;
+ goto end;
}
if( p_handler->pf_handler.smpl )
default:
/* unknown/unexpected xml node */
msg_Err( p_demux, "unexpected xml node %i", i_node );
- FREE_ATT();
- return false;
+ goto end;
}
- FREE_NAME();
}
- return false;
+
+end:
+ free( psz_name );
+ free( psz_value );
+ return b_ret;
}
/**
static bool parse_tracklist_node COMPLEX_INTERFACE
{
VLC_UNUSED(psz_element);
- char *psz_name=NULL;
- int i_node;
+ char *psz_name;
int i_ntracks = 0;
/* now parse the <track>s */
while( xml_ReaderRead( p_xml_reader ) == 1 )
{
- i_node = xml_ReaderNodeType( p_xml_reader );
+ int i_node = xml_ReaderNodeType( p_xml_reader );
if( i_node == XML_READER_STARTELEM )
{
- psz_name = xml_ReaderName( p_xml_reader );
- if( !psz_name )
+ char *psz_eltname = xml_ReaderName( p_xml_reader );
+ if( !psz_eltname )
{
msg_Err( p_demux, "unexpected end of xml data" );
- FREE_NAME();
+ free( psz_eltname );
return false;
}
- if( strcmp( psz_name, "track") )
+ if( strcmp( psz_eltname, "track") )
{
msg_Err( p_demux, "unexpected child of <trackList>: <%s>",
- psz_name );
- FREE_NAME();
+ psz_eltname );
+ free( psz_eltname );
return false;
}
- FREE_NAME();
+ free( psz_eltname );
/* parse the track data in a separate function */
- if( parse_track_node( p_demux, p_input_item,
- p_xml_reader,"track" ) == true )
+ if( parse_track_node( p_demux, p_input_node,
+ p_xml_reader, "track" ) )
i_ntracks++;
}
else if( i_node == XML_READER_ENDELEM )
if( xml_ReaderNodeType( p_xml_reader ) != XML_READER_ENDELEM )
{
msg_Err( p_demux, "there's a missing </trackList>" );
- FREE_NAME();
return false;
}
psz_name = xml_ReaderName( p_xml_reader );
if( !psz_name || strcmp( psz_name, "trackList" ) )
{
msg_Err( p_demux, "expected: </trackList>, found: </%s>", psz_name );
- FREE_NAME();
+ free( psz_name );
return false;
}
- FREE_NAME();
+ free( psz_name );
msg_Dbg( p_demux, "parsed %i tracks successfully", i_ntracks );
-
return true;
}
*/
static bool parse_track_node COMPLEX_INTERFACE
{
- input_item_t *p_new_input = NULL;
- int i_node;
- char *psz_name=NULL;
- char *psz_value=NULL;
- xml_elem_hnd_t *p_handler=NULL;
+ input_item_t *p_input_item = p_input_node->p_item;
+ char *psz_name = NULL;
+ char *psz_value = NULL;
+ xml_elem_hnd_t *p_handler = NULL;
+ demux_sys_t *p_sys = p_demux->p_sys;
+ bool b_ret = false;
xml_elem_hnd_t track_elements[] =
{ {"location", SIMPLE_CONTENT, {NULL} },
{NULL, UNKNOWN_CONTENT, {NULL} }
};
+ input_item_t *p_new_input = input_item_New( p_demux, NULL, NULL );
+ input_item_node_t *p_new_node = input_item_node_Create( p_new_input );
+
+ if( !p_new_input )
+ {
+ /* malloc has failed for input_item_New, so bailout early */
+ return false;
+ }
+
+ /* reset i_track_id */
+ p_sys->i_track_id = -1;
+
while( xml_ReaderRead( p_xml_reader ) == 1 )
{
- i_node = xml_ReaderNodeType( p_xml_reader );
+ int i_node = xml_ReaderNodeType( p_xml_reader );
switch( i_node )
{
case XML_READER_NONE:
if( !psz_name || !*psz_name )
{
msg_Err( p_demux, "invalid xml stream" );
- FREE_ATT();
- return false;
+ goto end;
}
/* choose handler */
for( p_handler = track_elements;
if( !p_handler->name )
{
msg_Err( p_demux, "unexpected element <%s>", psz_name );
- FREE_ATT();
- return false;
+ goto end;
}
FREE_NAME();
/* complex content is parsed in a separate function */
if( p_handler->type == COMPLEX_CONTENT )
{
- if( !p_new_input )
- {
- msg_Err( p_demux,
- "at <%s> level no new item has been allocated",
- p_handler->name );
- FREE_ATT();
- return false;
- }
- if( p_handler->pf_handler.cmplx( p_demux,
- p_new_input,
- p_xml_reader,
- p_handler->name ) )
+ FREE_VALUE();
+
+ bool b_res = p_handler->pf_handler.cmplx( p_demux,
+ p_new_node,
+ p_xml_reader,
+ p_handler->name );
+ if( b_res )
{
p_handler = NULL;
- FREE_ATT();
}
else
{
- FREE_ATT();
return false;
}
}
case XML_READER_TEXT:
/* simple element content */
- FREE_ATT();
+ free( psz_value );
psz_value = xml_ReaderValue( p_xml_reader );
if( !psz_value )
{
msg_Err( p_demux, "invalid xml stream" );
- FREE_ATT();
- return false;
+ goto end;
}
break;
if( !psz_name )
{
msg_Err( p_demux, "invalid xml stream" );
- FREE_ATT();
- return false;
+ goto end;
}
+
/* leave if the current parent node <track> is terminated */
if( !strcmp( psz_name, psz_element ) )
{
- FREE_ATT();
- if( p_demux->p_sys->i_identifier <
- p_demux->p_sys->i_tracklist_entries )
+ free( psz_name );
+ free( psz_value );
+
+ /* Make sure we have a URI */
+ char *psz_uri = input_item_GetURI( p_new_input );
+ if( !psz_uri )
{
- p_demux->p_sys->pp_tracklist[
- p_demux->p_sys->i_identifier ] = p_new_input;
+ input_item_SetURI( p_new_input, "vlc://nop" );
}
- else
+ free( psz_uri );
+
+ if( p_sys->i_track_id < 0 )
{
- if( p_demux->p_sys->i_identifier >
- p_demux->p_sys->i_tracklist_entries )
- {
- p_demux->p_sys->i_tracklist_entries =
- p_demux->p_sys->i_identifier;
- }
- INSERT_ELEM( p_demux->p_sys->pp_tracklist,
- p_demux->p_sys->i_tracklist_entries,
- p_demux->p_sys->i_tracklist_entries,
- p_new_input );
+ input_item_node_AppendNode( p_input_node, p_new_node );
+ vlc_gc_decref( p_new_input );
+ return true;
+ }
+
+ if( p_sys->i_track_id >= p_sys->i_tracklist_entries )
+ {
+ input_item_t **pp;
+ pp = realloc( p_sys->pp_tracklist,
+ (p_sys->i_track_id + 1) * sizeof(*pp) );
+ if( !pp )
+ return false;
+ p_sys->pp_tracklist = pp;
+ while( p_sys->i_track_id >= p_sys->i_tracklist_entries )
+ pp[p_sys->i_tracklist_entries++] = NULL;
}
+
+ p_sys->pp_tracklist[ p_sys->i_track_id ] = p_new_input;
return true;
}
/* there MUST have been a start tag for that element name */
{
msg_Err( p_demux, "there's no open element left for <%s>",
psz_name );
- FREE_ATT();
- return false;
+ goto end;
}
/* special case: location */
if( !strcmp( p_handler->name, "location" ) )
{
- char *psz_uri=NULL;
- /* there MUST NOT be an item */
- if( p_new_input )
- {
- msg_Err( p_demux, "item <%s> already created",
- psz_name );
- FREE_ATT();
- return false;
- }
- psz_uri = decode_URI_duplicate( psz_value );
-
- if( psz_uri )
+ if( psz_value == NULL )
+ input_item_SetURI( p_new_input, "vlc://nop" );
+ else
+ /* FIXME: This is broken. Scheme-relative (//...) locations
+ * and anchors (#...) are not resolved correctly. Also,
+ * host-relative (/...) and directory-relative locations
+ * ("relative path" in vernacular) should be resolved.
+ * Last, psz_base should default to the XSPF resource
+ * location if missing (not the current working directory).
+ * -- Courmisch */
+ if( p_sys->psz_base && !strstr( psz_value, "://" ) )
{
- if( p_demux->p_sys->psz_base &&
- !strstr( psz_uri, "://" ) )
+ char* psz_tmp;
+ if( asprintf( &psz_tmp, "%s%s", p_sys->psz_base,
+ psz_value ) == -1 )
{
- char* psz_tmp = malloc(
- strlen(p_demux->p_sys->psz_base) +
- strlen(psz_uri) +1 );
- if( !psz_tmp )
- return false;
- sprintf( psz_tmp, "%s%s",
- p_demux->p_sys->psz_base, psz_uri );
- free( psz_uri );
- psz_uri = psz_tmp;
+ goto end;
}
- p_new_input = input_ItemNewExt( p_demux, psz_uri,
- NULL, 0, NULL, -1 );
- free( psz_uri );
- input_ItemCopyOptions( p_input_item, p_new_input );
- psz_uri = NULL;
- FREE_ATT();
- p_handler = NULL;
+ input_item_SetURI( p_new_input, psz_tmp );
+ free( psz_tmp );
}
else
- {
- FREE_ATT();
- return false;
- }
- }
- else if( !strcmp( p_handler->name, "identifier" ) )
- {
- p_demux->p_sys->i_identifier = atoi( psz_value );
+ input_item_SetURI( p_new_input, psz_value );
+ input_item_CopyOptions( p_input_item, p_new_input );
}
else
{
/* there MUST be an item */
- if( !p_new_input )
- {
- msg_Err( p_demux, "item not yet created at <%s>",
- psz_name );
- FREE_ATT();
- return false;
- }
if( p_handler->pf_handler.smpl )
{
p_handler->pf_handler.smpl( p_new_input,
default:
/* unknown/unexpected xml node */
msg_Err( p_demux, "unexpected xml node %i", i_node );
- FREE_ATT();
- return false;
+ goto end;
}
- FREE_NAME();
}
msg_Err( p_demux, "unexpected end of xml data" );
- FREE_ATT();
- return false;
+
+end:
+ free( psz_name );
+ free( psz_value );
+ return b_ret;
}
/**
if( !psz_name || !psz_value || !p_input )
return false;
-
/* re-convert xml special characters inside psz_value */
resolve_xml_special_chars( psz_value );
}
else if( !strcmp( psz_name, "image" ) )
{
- char *psz_uri = decode_URI_duplicate( psz_value );
- input_item_SetArtURL( p_input, psz_uri );
- free( psz_uri );
+ input_item_SetArtURL( p_input, psz_value );
}
return true;
}
/**
- * \brief handles the <option> elements
+ * \brief handles the <vlc:option> elements
*/
static bool set_option SIMPLE_INTERFACE
{
/* re-convert xml special characters inside psz_value */
resolve_xml_special_chars( psz_value );
-
- input_ItemAddOpt( p_input, psz_value, 0 );
-
+
+ input_item_AddOption( p_input, psz_value, 0 );
+
return true;
}
*/
static bool parse_extension_node COMPLEX_INTERFACE
{
+ input_item_t *p_input_item = p_input_node->p_item;
char *psz_name = NULL;
char *psz_value = NULL;
char *psz_title = NULL;
char *psz_application = NULL;
int i_node;
+ bool b_release_input_item = false;
xml_elem_hnd_t *p_handler = NULL;
input_item_t *p_new_input = NULL;
xml_elem_hnd_t pl_elements[] =
- { {"node", COMPLEX_CONTENT, {.cmplx = parse_extension_node} },
- {"item", COMPLEX_CONTENT, {.cmplx = parse_extitem_node} },
- {"option", SIMPLE_CONTENT, {.smpl = set_option} },
+ { {"vlc:node", COMPLEX_CONTENT, {.cmplx = parse_extension_node} },
+ {"vlc:item", COMPLEX_CONTENT, {.cmplx = parse_extitem_node} },
+ {"vlc:id", SIMPLE_CONTENT, {NULL} },
+ {"vlc:option", SIMPLE_CONTENT, {.smpl = set_option} },
{NULL, UNKNOWN_CONTENT, {NULL} }
};
psz_value = xml_ReaderValue( p_xml_reader );
if( !psz_name || !psz_value )
{
- msg_Err( p_demux, "invalid xml stream @ <node>" );
+ msg_Err( p_demux, "invalid xml stream @ <vlc:node>" );
FREE_ATT();
return false;
}
if( !strcmp( psz_name, "title" ) )
{
resolve_xml_special_chars( psz_value );
- psz_title = strdup( psz_value );
+ psz_title = psz_value;
}
/* extension attribute: application */
else if( !strcmp( psz_name, "application" ) )
{
- psz_application = strdup( psz_value );
+ psz_application = psz_value;
}
/* unknown attribute */
else
- msg_Warn( p_demux, "invalid <%s> attribute:\"%s\"", psz_element, psz_name );
-
- FREE_ATT();
+ {
+ msg_Warn( p_demux, "invalid <%s> attribute:\"%s\"", psz_element,
+ psz_name );
+ FREE_VALUE();
+ }
+ FREE_NAME();
+ psz_value = NULL;
}
/* attribute title is mandatory except for <extension> */
- if( !strcmp( psz_element, "node" ) )
+ if( !strcmp( psz_element, "vlc:node" ) )
{
if( !psz_title )
{
- msg_Warn( p_demux, "<node> requires \"title\" attribute" );
+ msg_Warn( p_demux, "<vlc:node> requires \"title\" attribute" );
return false;
}
- p_new_input = input_ItemNewWithType( VLC_OBJECT( p_demux ), "vlc://nop",
- psz_title, 0, NULL, -1, ITEM_TYPE_DIRECTORY );
+ p_new_input = input_item_NewWithType( VLC_OBJECT( p_demux ),
+ "vlc://nop", psz_title, 0, NULL, 0, -1,
+ ITEM_TYPE_DIRECTORY );
if( p_new_input )
{
- input_ItemAddSubItem( p_input_item, p_new_input );
+ p_input_node =
+ input_item_node_AppendItem( p_input_node, p_new_input );
p_input_item = p_new_input;
- vlc_gc_decref( p_new_input );
+ b_release_input_item = true;
}
free( psz_title );
}
msg_Warn( p_demux, "<extension> requires \"application\" attribute" );
return false;
}
+ /* Skip the extension if the application is not vlc
+ This will skip all children of the current node */
else if( strcmp( psz_application, "http://www.videolan.org/vlc/playlist/0" ) )
{
msg_Dbg( p_demux, "Skipping \"%s\" extension tag", psz_application );
free( psz_application );
- return false;
+ /* Skip all children */
+ while( xml_ReaderRead( p_xml_reader ) == 1 )
+ {
+ if( xml_ReaderNodeType( p_xml_reader ) == XML_READER_ENDELEM )
+ {
+ char *psz_name = xml_ReaderName( p_xml_reader );
+ if( !strcmp( psz_name, "extension" ) )
+ {
+ free( psz_name );
+ break;
+ }
+ msg_Dbg( p_demux, "\tskipping \"%s\" extension child", psz_name );
+ free( psz_name );
+ }
+ }
+ return true;
}
}
free( psz_application );
+
/* parse the child elements */
while( xml_ReaderRead( p_xml_reader ) == 1 )
{
{
msg_Err( p_demux, "invalid xml stream" );
FREE_ATT();
+ if( b_release_input_item ) vlc_gc_decref( p_new_input );
return false;
}
/* choose handler */
{
msg_Err( p_demux, "unexpected element <%s>", psz_name );
FREE_ATT();
+ if( b_release_input_item ) vlc_gc_decref( p_new_input );
return false;
}
FREE_NAME();
if( p_handler->type == COMPLEX_CONTENT )
{
if( p_handler->pf_handler.cmplx( p_demux,
- p_input_item,
+ p_input_node,
p_xml_reader,
p_handler->name ) )
{
else
{
FREE_ATT();
+ if( b_release_input_item ) vlc_gc_decref( p_new_input );
return false;
}
}
{
msg_Err( p_demux, "invalid xml stream" );
FREE_ATT();
+ if( b_release_input_item ) vlc_gc_decref( p_new_input );
return false;
}
break;
{
msg_Err( p_demux, "invalid xml stream" );
FREE_ATT();
+ if( b_release_input_item ) vlc_gc_decref( p_new_input );
return false;
}
/* leave if the current parent node is terminated */
if( !strcmp( psz_name, psz_element ) )
{
FREE_ATT();
+ if( b_release_input_item ) vlc_gc_decref( p_new_input );
return true;
}
/* there MUST have been a start tag for that element name */
msg_Err( p_demux, "there's no open element left for <%s>",
psz_name );
FREE_ATT();
+ if( b_release_input_item ) vlc_gc_decref( p_new_input );
return false;
}
- if( p_handler->pf_handler.smpl )
+ /* special tag <vlc:id> */
+ if( !strcmp( p_handler->name, "vlc:id" ) )
+ {
+ p_demux->p_sys->i_track_id = atoi( psz_value );
+ }
+ else if( p_handler->pf_handler.smpl )
{
p_handler->pf_handler.smpl( p_input_item, p_handler->name,
psz_value );
/* unknown/unexpected xml node */
msg_Err( p_demux, "unexpected xml node %i", i_node );
FREE_ATT();
+ if( b_release_input_item ) vlc_gc_decref( p_new_input );
return false;
}
FREE_NAME();
}
+ if( b_release_input_item ) vlc_gc_decref( p_new_input );
return false;
}
{
VLC_UNUSED(psz_element);
input_item_t *p_new_input = NULL;
- char *psz_name = NULL;
- char *psz_value = NULL;
- int i_href = -1;
+ int i_tid = -1;
/* read all extension item attributes */
while( xml_ReaderNextAttr( p_xml_reader ) == VLC_SUCCESS )
{
- psz_name = xml_ReaderName( p_xml_reader );
- psz_value = xml_ReaderValue( p_xml_reader );
+ char *psz_name = xml_ReaderName( p_xml_reader );
+ char *psz_value = xml_ReaderValue( p_xml_reader );
if( !psz_name || !psz_value )
{
- msg_Err( p_demux, "invalid xml stream @ <item>" );
- FREE_ATT();
+ msg_Err( p_demux, "invalid xml stream @ <vlc:item>" );
+ free( psz_name );
+ free( psz_value );
return false;
}
/* attribute: href */
- if( !strcmp( psz_name, "href" ) )
+ if( !strcmp( psz_name, "tid" ) )
{
- i_href = atoi( psz_value );
+ i_tid = atoi( psz_value );
}
/* unknown attribute */
else
- msg_Warn( p_demux, "invalid <item> attribute:\"%s\"", psz_name);
+ msg_Warn( p_demux, "invalid <vlc:item> attribute:\"%s\"", psz_name);
- FREE_ATT();
+ free( psz_name );
+ free( psz_value );
}
/* attribute href is mandatory */
- if( i_href < 0 )
+ if( i_tid < 0 )
{
- msg_Warn( p_demux, "<item> requires \"href\" attribute" );
+ msg_Warn( p_demux, "<vlc:item> requires \"tid\" attribute" );
return false;
}
- if( i_href >= p_demux->p_sys->i_tracklist_entries )
+ if( i_tid >= p_demux->p_sys->i_tracklist_entries )
{
- msg_Warn( p_demux, "invalid \"href\" attribute" );
+ msg_Warn( p_demux, "invalid \"tid\" attribute" );
return false;
}
- p_new_input = p_demux->p_sys->pp_tracklist[ i_href ];
+ p_new_input = p_demux->p_sys->pp_tracklist[ i_tid ];
if( p_new_input )
{
- input_ItemAddSubItem( p_input_item, p_new_input );
+ input_item_node_AppendItem( p_input_node, p_new_input );
vlc_gc_decref( p_new_input );
- p_demux->p_sys->pp_tracklist[i_href] = NULL;
+ p_demux->p_sys->pp_tracklist[i_tid] = NULL;
}
/* kludge for #1293 - XTAG sends ENDELEM for self closing tag */
*/
static bool skip_element COMPLEX_INTERFACE
{
- VLC_UNUSED(p_demux); VLC_UNUSED(p_input_item);
- char *psz_endname;
+ VLC_UNUSED(p_demux); VLC_UNUSED(p_input_node);
while( xml_ReaderRead( p_xml_reader ) == 1 )
{
if( xml_ReaderNodeType( p_xml_reader ) == XML_READER_ENDELEM )
{
- psz_endname = xml_ReaderName( p_xml_reader );
+ char *psz_endname = xml_ReaderName( p_xml_reader );
if( !psz_endname )
return false;
if( !strcmp( psz_element, psz_endname ) )
projects / vlc / blobdiff