]> git.sesse.net Git - vlc/blobdiff - modules/services_discovery/podcast.c
Fix [9589813471d0f5525789b7cf7165e48d177cbad6] by using var_Create (Pointed out by...
[vlc] / modules / services_discovery / podcast.c
index 06674f9bb99e81a404080cb347361e12ba898cdc..b64243e569b4aff8e05306262a451b85619b7a42 100644 (file)
  *
  * 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., 59 Temple Place - Suite 330, Boston, MA  02111, USA.
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
  *****************************************************************************/
 
 /*****************************************************************************
  * Includes
  *****************************************************************************/
-#include <stdlib.h>                                      /* malloc(), free() */
 
-#include <vlc/vlc.h>
-#include <vlc/intf.h>
-
-#include <vlc/input.h>
+#ifdef HAVE_CONFIG_H
+# include "config.h"
+#endif
 
-#include "network.h"
+#include <vlc_common.h>
+#include <vlc_plugin.h>
+#include <vlc_playlist.h>
+#include <vlc_network.h>
 
 #include <errno.h>                                                 /* ENOMEM */
 
  *****************************************************************************/
 
 /* Callbacks */
-    static int  Open ( vlc_object_t * );
-    static void Close( vlc_object_t * );
+static int  Open ( vlc_object_t * );
+static void Close( vlc_object_t * );
 
 #define URLS_TEXT N_("Podcast URLs list")
-#define URLS_LONGTEXT N_("Podcast '|' (pipe) seperated URLs list")
+#define URLS_LONGTEXT N_("Enter the list of podcasts to retrieve, " \
+                         "separated by '|' (pipe)." )
 
 vlc_module_begin();
     set_shortname( "Podcast");
-    set_description( _("Podcast Service Discovery") );
+    set_description( N_("Podcasts") );
     set_category( CAT_PLAYLIST );
     set_subcategory( SUBCAT_PLAYLIST_SD );
 
     add_string( "podcast-urls", NULL, NULL,
-                URLS_TEXT, URLS_LONGTEXT, VLC_FALSE );
+                URLS_TEXT, URLS_LONGTEXT, false );
+        change_autosave();
 
     set_capability( "services_discovery", 0 );
     set_callbacks( Open, Close );
@@ -79,19 +82,22 @@ vlc_module_end();
 struct services_discovery_sys_t
 {
     /* playlist node */
-    playlist_item_t *p_node;
-    input_thread_t *p_input;
+    input_thread_t **pp_input;
+    int i_input;
 
     char **ppsz_urls;
     int i_urls;
+
+    bool b_update;
 };
 
 /*****************************************************************************
  * Local prototypes
  *****************************************************************************/
-
-/* Main functions */
-    static void Run    ( services_discovery_t *p_intf );
+static void Run( services_discovery_t *p_intf );
+static int UrlsChange( vlc_object_t *, char const *, vlc_value_t,
+                       vlc_value_t, void * );
+static void ParseUrls( services_discovery_t *p_sd, char *psz_urls );
 
 /*****************************************************************************
  * Open: initialize and create stuff
@@ -101,81 +107,20 @@ static int Open( vlc_object_t *p_this )
     services_discovery_t *p_sd = ( services_discovery_t* )p_this;
     services_discovery_sys_t *p_sys  = malloc(
                                     sizeof( services_discovery_sys_t ) );
+    if( !p_sys )
+        return VLC_ENOMEM;
 
-    vlc_value_t         val;
-    playlist_t          *p_playlist;
-    playlist_view_t     *p_view;
-    playlist_item_t     *p_item;
-
-    int i, j;
-    char *psz_buf;
-    char *psz_tmp = psz_buf = var_CreateGetString( p_sd, "podcast-urls" );
-
-    i = 0;
-    p_sys->i_urls = 1;
-    while( psz_buf[i] != 0 )
-        if( psz_buf[i++] == '|' )
-            p_sys->i_urls++;
-
-    p_sys->ppsz_urls = (char **)malloc( p_sys->i_urls * sizeof( char * ) );
-
-    i = 0;
-    j = 0;
-    while( psz_buf[i] != 0 )
-    {
-        if( psz_buf[i] == '|' )
-        {
-            psz_buf[i] = 0;
-            p_sys->ppsz_urls[j] = strdup( psz_tmp );
-            i++;
-            j++;
-            psz_tmp = psz_buf+i;
-        }
-        else
-            i++;
-    }
-    p_sys->ppsz_urls[j] = strdup( psz_tmp );
-    free( psz_buf );
+    p_sys->i_urls = 0;
+    p_sys->ppsz_urls = NULL;
+    p_sys->i_input = 0;
+    p_sys->pp_input = NULL;
+    p_sys->b_update = true;
 
     p_sd->pf_run = Run;
     p_sd->p_sys  = p_sys;
 
-    /* Create our playlist node */
-    p_playlist = (playlist_t *)vlc_object_find( p_sd, VLC_OBJECT_PLAYLIST,
-                                                FIND_ANYWHERE );
-    if( !p_playlist )
-    {
-        msg_Warn( p_sd, "unable to find playlist, cancelling");
-        return VLC_EGENERIC;
-    }
-
-    p_view = playlist_ViewFind( p_playlist, VIEW_CATEGORY );
-    p_sys->p_node = playlist_NodeCreate( p_playlist, VIEW_CATEGORY,
-                                         _("Podcast"), p_view->p_root );
-    for( i = 0; i < p_sys->i_urls; i++ )
-    {
-        asprintf( &psz_buf, "%s", p_sys->ppsz_urls[i] );
-        p_item = playlist_ItemNew( p_playlist, psz_buf,
-                                         p_sys->ppsz_urls[i] );
-        free( psz_buf );
-        playlist_ItemAddOption( p_item, "demux=podcast" );
-        playlist_NodeAddItem( p_playlist, p_item,
-                              p_sys->p_node->pp_parents[0]->i_view,
-                              p_sys->p_node, PLAYLIST_APPEND,
-                              PLAYLIST_END );
-
-        /* We need to declare the parents of the node as the same of the
-         * parent's ones */
-        playlist_CopyParents( p_sys->p_node, p_item );
-        p_sys->p_input = input_CreateThread( p_playlist, &p_item->input );
-    }
-
-
-    p_sys->p_node->i_flags |= PLAYLIST_RO_FLAG;
-    val.b_bool = VLC_TRUE;
-    var_Set( p_playlist, "intf-change", val );
-
-    vlc_object_release( p_playlist );
+    /* Give us a name */
+    services_discovery_SetLocalizedName( p_sd, _("Podcasts") );
 
     return VLC_SUCCESS;
 }
@@ -187,22 +132,17 @@ static void Close( vlc_object_t *p_this )
 {
     services_discovery_t *p_sd = ( services_discovery_t* )p_this;
     services_discovery_sys_t *p_sys  = p_sd->p_sys;
-    playlist_t *p_playlist =  (playlist_t *) vlc_object_find( p_sd,
-                                 VLC_OBJECT_PLAYLIST, FIND_ANYWHERE );
     int i;
-    if( p_sd->p_sys->p_input )
+    for( i = 0; i < p_sys->i_input; i++ )
     {
-        input_StopThread( p_sd->p_sys->p_input );
-        input_DestroyThread( p_sd->p_sys->p_input );
-        vlc_object_detach( p_sd->p_sys->p_input );
-        vlc_object_destroy( p_sd->p_sys->p_input );
-        p_sd->p_sys->p_input = NULL;
-    }
-    if( p_playlist )
-    {
-        playlist_NodeDelete( p_playlist, p_sys->p_node, VLC_TRUE, VLC_TRUE );
-        vlc_object_release( p_playlist );
+        if( p_sd->p_sys->pp_input[i] )
+        {
+            input_StopThread( p_sd->p_sys->pp_input[i] );
+            vlc_object_release( p_sd->p_sys->pp_input[i] );
+            p_sd->p_sys->pp_input[i] = NULL;
+        }
     }
+    free( p_sd->p_sys->pp_input );
     for( i = 0; i < p_sys->i_urls; i++ ) free( p_sys->ppsz_urls[i] );
     free( p_sys->ppsz_urls );
     free( p_sys );
@@ -213,17 +153,80 @@ static void Close( vlc_object_t *p_this )
  *****************************************************************************/
 static void Run( services_discovery_t *p_sd )
 {
-    while( !p_sd->b_die )
+    services_discovery_sys_t *p_sys  = p_sd->p_sys;
+
+    /* Launch the callback associated with this variable */
+    var_Create( p_sd, "podcast-urls", VLC_VAR_STRING | VLC_VAR_DOINHERIT );
+    var_AddCallback( p_sd, "podcast-urls", UrlsChange, p_sys );
+
+    while( vlc_object_alive (p_sd) )
+    {
+        int i;
+        if( p_sys->b_update == true )
+        {
+            msg_Dbg( p_sd, "Update required" );
+            char* psz_urls = var_GetNonEmptyString( p_sd, "podcast-urls" );
+            if( psz_urls != NULL )
+                ParseUrls( p_sd, psz_urls );
+            free( psz_urls );
+            p_sys->b_update = false;
+        }
+
+        for( i = 0; i < p_sd->p_sys->i_input; i++ )
+        {
+            if( p_sd->p_sys->pp_input[i]->b_eof
+                || p_sd->p_sys->pp_input[i]->b_error )
+            {
+                input_StopThread( p_sd->p_sys->pp_input[i] );
+                vlc_object_release( p_sd->p_sys->pp_input[i] );
+                p_sd->p_sys->pp_input[i] = NULL;
+                REMOVE_ELEM( p_sys->pp_input, p_sys->i_input, i );
+                i--;
+            }
+        }
+        msleep( 500 );
+    }
+}
+
+static int UrlsChange( vlc_object_t *p_this, char const *psz_var,
+                       vlc_value_t oldval, vlc_value_t newval,
+                       void *p_data )
+{
+    VLC_UNUSED(p_this); VLC_UNUSED(psz_var); VLC_UNUSED(oldval);
+    VLC_UNUSED(newval);
+    services_discovery_sys_t *p_sys  = (services_discovery_sys_t *)p_data;
+    p_sys->b_update = true;
+    return VLC_SUCCESS;
+}
+
+static void ParseUrls( services_discovery_t *p_sd, char *psz_urls )
+{
+    services_discovery_sys_t *p_sys = p_sd->p_sys;
+    for( ;; )
     {
-        if( p_sd->p_sys->p_input &&
-            ( p_sd->p_sys->p_input->b_eof || p_sd->p_sys->p_input->b_error ) )
+        int i;
+        char *psz_tok = strchr( psz_urls, '|' );
+        if( psz_tok ) *psz_tok = '\0';
+        for( i = 0; i < p_sys->i_urls; i++ )
+            if( !strcmp( psz_urls, p_sys->ppsz_urls[i] ) )
+                break;
+        if( i == p_sys->i_urls )
         {
-            input_StopThread( p_sd->p_sys->p_input );
-            input_DestroyThread( p_sd->p_sys->p_input );
-            vlc_object_detach( p_sd->p_sys->p_input );
-            vlc_object_destroy( p_sd->p_sys->p_input );
-            p_sd->p_sys->p_input = NULL;
+            /* Only add new urls.
+             * FIXME: We don't delete urls which have been removed from
+             * the config since we don't have a way to know which inputs
+             * they spawned */
+            input_item_t *p_input;
+            INSERT_ELEM( p_sys->ppsz_urls, p_sys->i_urls, p_sys->i_urls,
+                         strdup( psz_urls ) );
+            p_input = input_ItemNewExt( p_sd, psz_urls,
+                                        psz_urls, 0, NULL, -1 );
+            input_ItemAddOption( p_input, "demux=podcast" );
+            services_discovery_AddItem( p_sd, p_input, NULL /* no cat */ );
+            INSERT_ELEM( p_sys->pp_input, p_sys->i_input, p_sys->i_input,
+                         input_CreateThread( p_sd, p_input ) );
         }
-        msleep( 100000 );
+        if( psz_tok )  psz_urls = psz_tok+1;
+        else           return;
     }
 }