[vlc-devel] commit: Revert "directory: Add XSPF node-extension support to our directory module." ( Rémi Denis-Courmont )

git version control git at videolan.org
Sun Feb 8 10:12:05 CET 2009


vlc | branch: master | Rémi Denis-Courmont <rdenis at simphalempin.com> | Sun Feb  8 11:11:30 2009 +0200| [bc7bf1b597db73adf4018dfdbc8b58a2d7ed74e9] | committer: Rémi Denis-Courmont 

Revert "directory: Add XSPF node-extension support to our directory module."

This reverts commit 8f7056b536e341fe8f07cfec4067e53e93c59708.
This broke the XML parser (fixes #2500).

Conflicts:

	modules/access/directory.c

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=bc7bf1b597db73adf4018dfdbc8b58a2d7ed74e9
---

 modules/access/directory.c |   66 +++++--------------------------------------
 1 files changed, 8 insertions(+), 58 deletions(-)

diff --git a/modules/access/directory.c b/modules/access/directory.c
index 379c1a1..ff4309c 100644
--- a/modules/access/directory.c
+++ b/modules/access/directory.c
@@ -125,8 +125,6 @@ struct access_sys_t
     DIR *handle;
     char *ignored_exts;
     int mode;
-    int i_item_count;
-    char *psz_xspf_extension;
 };
 
 static block_t *Block( access_t * );
@@ -173,8 +171,6 @@ static int Open( vlc_object_t *p_this )
     p_sys->current = NULL;
     p_sys->handle = handle;
     p_sys->ignored_exts = var_CreateGetString (p_access, "ignore-filetypes");
-    p_sys->i_item_count = 0;
-    p_sys->psz_xspf_extension = strdup( "" );
 
     /* Handle mode */
     char *psz = var_CreateGetString( p_access, "recursive" );
@@ -215,7 +211,6 @@ static void Close( vlc_object_t * p_this )
     }
     if (p_sys->handle != NULL)
         closedir (p_sys->handle); /* corner case,:Block() not called ever */
-    free (p_sys->psz_xspf_extension);
     free (p_sys->ignored_exts);
     free (p_sys);
 }
@@ -269,7 +264,7 @@ static block_t *Block (access_t *p_access)
     {   /* Startup: send the XSPF header */
         static const char header[] =
             "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n"
-            "<playlist version=\"1\" xmlns=\"http://xspf.org/ns/0/\" xmlns:vlc=\"http://www.videolan.org/vlc/playlist/ns/0/\">\n"
+            "<playlist version=\"1\" xmlns=\"http://xspf.org/ns/0/\">\n"
             " <trackList>\n";
         block_t *block = block_Alloc (sizeof (header) - 1);
         if (!block)
@@ -310,36 +305,17 @@ static block_t *Block (access_t *p_access)
 
         if (p_sys->current == NULL)
         {   /* End of XSPF playlist */
-            char *footer;
-            int len = asprintf( &footer, " </trackList>\n" \
-                " <extension application=\"http://www.videolan.org/vlc/playlist/0\">\n" \
-                "%s" \
-                " </extension>\n" \
-                "</playlist>\n", p_sys->psz_xspf_extension );
-            if( len < 0 )
-                goto fatal;
+            static const char footer[] =
+                " </trackList>\n"
+                "</playlist>\n";
 
-            block_t *block = block_Alloc ( len );
+            block_t *block = block_Alloc (sizeof (footer) - 1);
             if (!block)
                 goto fatal;
-            memcpy (block->p_buffer, footer, len);
-            free( footer );
+            memcpy (block->p_buffer, footer, sizeof (footer) - 1);
             p_access->info.b_eof = true;
             return block;
         }
-        else
-        {
-            /* This was the end of a "subnode" */
-            /* Write the ID to the extension */
-            char *old_xspf_extension = p_sys->psz_xspf_extension;
-            if (old_xspf_extension == NULL)
-                goto fatal;
-
-            int len2 = asprintf( &p_sys->psz_xspf_extension, "%s  </vlc:node>\n", old_xspf_extension );
-            if (len2 == -1)
-                goto fatal;
-            free( old_xspf_extension );
-        }
         return NULL;
     }
 
@@ -377,17 +353,6 @@ static block_t *Block (access_t *p_access)
                 return NULL;
             }
             p_sys->current = sub;
-
-            /* Add node to xspf extension */
-            char *old_xspf_extension = p_sys->psz_xspf_extension;
-            if (old_xspf_extension == NULL)
-                goto fatal;
-
-            int len2 = asprintf( &p_sys->psz_xspf_extension, "%s  <vlc:node title=\"%s\">\n", old_xspf_extension, entry );
-            if (len2 == -1)
-                goto fatal;
-            free( old_xspf_extension );
-
             return NULL;
         }
         else
@@ -423,27 +388,12 @@ static block_t *Block (access_t *p_access)
     if (encoded == NULL)
         goto fatal;
     int len = asprintf (&entry,
-                        "  <track><location>file://%s/%s</location>\n" \
-                        "   <extension application=\"http://www.videolan.org/vlc/playlist/0\">\n" \
-                        "    <vlc:id>%d</vlc:id>\n" \
-                        "   </extension>\n" \
-                        "  </track>\n",
-                        current->uri, encoded, p_sys->i_item_count++);
+                        "  <track><location>file://%s/%s</location></track>\n",
+                        current->uri, encoded);
     free (encoded);
     if (len == -1)
         goto fatal;
 
-    /* Write the ID to the extension */
-    char *old_xspf_extension = p_sys->psz_xspf_extension;
-    if (old_xspf_extension == NULL)
-        goto fatal;
-
-    int len2 = asprintf( &p_sys->psz_xspf_extension, "%s   <vlc:item tid=\"%i\" />\n",
-                            old_xspf_extension, p_sys->i_item_count-1 );
-    if (len2 == -1)
-        goto fatal;
-    free( old_xspf_extension );
-
     /* TODO: new block allocator for malloc()ated data */
     block_t *block = block_Alloc (len);
     if (!block)




More information about the vlc-devel mailing list