/*****************************************************************************
* rss.c : rss/atom feed display video plugin for vlc
*****************************************************************************
- * Copyright (C) 2003-2006 the VideoLAN team
+ * Copyright (C) 2003-2006 VLC authors and VideoLAN
* $Id$
*
* Authors: Antoine Cellerier <dionoea -at- videolan -dot- org>
+ * RĂ©mi Duraffort <ivoire -at- videolan -dot- org>
*
- * 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.
*****************************************************************************/
/*****************************************************************************
#include <vlc_filter.h>
#include <vlc_block.h>
-#include <vlc_osd.h>
-#include <vlc_block.h>
#include <vlc_stream.h>
#include <vlc_xml.h>
#include <vlc_charset.h>
static void DestroyFilter( vlc_object_t * );
static subpicture_t *Filter( filter_t *, mtime_t );
-static int FetchRSS( filter_t * );
-static void FreeRSS( filter_t * );
+static struct rss_feed_t *FetchRSS( filter_t * );
+static void FreeRSS( struct rss_feed_t *, int );
static int ParseUrls( filter_t *, char * );
+static void Fetch( void * );
+
static const int pi_color_values[] = {
0xf0000000, 0x00000000, 0x00808080, 0x00C0C0C0,
0x00FFFFFF, 0x00800000, 0x00FF0000, 0x00FF00FF, 0x00FFFF00,
struct filter_sys_t
{
vlc_mutex_t lock;
+ vlc_timer_t timer; /* Timer to refresh the rss feeds */
+ bool b_fetched;
int i_xoff, i_yoff; /* offsets for the display string in the video window */
int i_pos; /* permit relative positioning (top, bottom, left, right, center) */
int i_feeds;
rss_feed_t *p_feeds;
- int i_ttl;
- time_t t_last_update;
bool b_images;
int i_title;
#define TITLE_TEXT N_("Title display mode")
#define TITLE_LONGTEXT N_("Title display mode. Default is 0 (hidden) if the feed has an image and feed images are enabled, 1 otherwise.")
+#define RSS_HELP N_("Display a RSS or ATOM Feed on your video")
+
static const int pi_pos_values[] = { 0, 1, 2, 4, 8, 5, 6, 9, 10 };
static const char *const ppsz_pos_descriptions[] =
{ N_("Center"), N_("Left"), N_("Right"), N_("Top"), N_("Bottom"),
* Module descriptor
*****************************************************************************/
vlc_module_begin ()
- set_capability( "sub filter", 1 )
- set_shortname( "RSS / Atom" )
+ set_capability( "sub source", 1 )
+ set_shortname( N_("RSS / Atom") )
+ set_help(RSS_HELP)
set_callbacks( CreateFilter, DestroyFilter )
set_category( CAT_VIDEO )
set_subcategory( SUBCAT_VIDEO_SUBPIC )
- add_string( CFG_PREFIX "urls", "rss", NULL, MSG_TEXT, MSG_LONGTEXT, false )
+ add_string( CFG_PREFIX "urls", NULL, MSG_TEXT, MSG_LONGTEXT, false )
set_section( N_("Position"), NULL )
- add_integer( CFG_PREFIX "x", 0, NULL, POSX_TEXT, POSX_LONGTEXT, true )
- add_integer( CFG_PREFIX "y", 0, NULL, POSY_TEXT, POSY_LONGTEXT, true )
- add_integer( CFG_PREFIX "position", -1, NULL, POS_TEXT, POS_LONGTEXT, false )
- change_integer_list( pi_pos_values, ppsz_pos_descriptions, NULL )
+ add_integer( CFG_PREFIX "x", 0, POSX_TEXT, POSX_LONGTEXT, true )
+ add_integer( CFG_PREFIX "y", 0, POSY_TEXT, POSY_LONGTEXT, true )
+ add_integer( CFG_PREFIX "position", -1, POS_TEXT, POS_LONGTEXT, false )
+ change_integer_list( pi_pos_values, ppsz_pos_descriptions )
set_section( N_("Font"), NULL )
/* 5 sets the default to top [1] left [4] */
- add_integer_with_range( CFG_PREFIX "opacity", 255, 0, 255, NULL,
+ add_integer_with_range( CFG_PREFIX "opacity", 255, 0, 255,
OPACITY_TEXT, OPACITY_LONGTEXT, false )
- add_integer( CFG_PREFIX "color", 0xFFFFFF, NULL, COLOR_TEXT, COLOR_LONGTEXT,
+ add_rgb( CFG_PREFIX "color", 0xFFFFFF, COLOR_TEXT, COLOR_LONGTEXT,
false )
- change_integer_list( pi_color_values, ppsz_color_descriptions, NULL )
- add_integer( CFG_PREFIX "size", -1, NULL, SIZE_TEXT, SIZE_LONGTEXT, false )
+ change_integer_list( pi_color_values, ppsz_color_descriptions )
+ add_integer( CFG_PREFIX "size", -1, SIZE_TEXT, SIZE_LONGTEXT, false )
+ change_integer_range( -1, 4096)
set_section( N_("Misc"), NULL )
- add_integer( CFG_PREFIX "speed", 100000, NULL, SPEED_TEXT, SPEED_LONGTEXT,
+ add_integer( CFG_PREFIX "speed", 100000, SPEED_TEXT, SPEED_LONGTEXT,
false )
- add_integer( CFG_PREFIX "length", 60, NULL, LENGTH_TEXT, LENGTH_LONGTEXT,
+ add_integer( CFG_PREFIX "length", 60, LENGTH_TEXT, LENGTH_LONGTEXT,
false )
- add_integer( CFG_PREFIX "ttl", 1800, NULL, TTL_TEXT, TTL_LONGTEXT, false )
- add_bool( CFG_PREFIX "images", 1, NULL, IMAGE_TEXT, IMAGE_LONGTEXT, false )
- add_integer( CFG_PREFIX "title", default_title, NULL, TITLE_TEXT, TITLE_LONGTEXT, false )
- change_integer_list( pi_title_modes, ppsz_title_modes, NULL )
+ add_integer( CFG_PREFIX "ttl", 1800, TTL_TEXT, TTL_LONGTEXT, false )
+ add_bool( CFG_PREFIX "images", true, IMAGE_TEXT, IMAGE_LONGTEXT, false )
+ add_integer( CFG_PREFIX "title", default_title, TITLE_TEXT, TITLE_LONGTEXT, false )
+ change_integer_list( pi_title_modes, ppsz_title_modes )
set_description( N_("RSS and Atom feed display") )
- add_shortcut( "rss" )
- add_shortcut( "atom" )
+ add_shortcut( "rss", "atom" )
vlc_module_end ()
static const char *const ppsz_filter_options[] = {
{
filter_t *p_filter = (filter_t *)p_this;
filter_sys_t *p_sys;
- int i_feed;
- int i_ret = VLC_ENOMEM;
char *psz_urls;
+ int i_ttl;
/* Allocate structure */
p_sys = p_filter->p_sys = malloc( sizeof( filter_sys_t ) );
config_ChainParse( p_filter, CFG_PREFIX, ppsz_filter_options,
p_filter->p_cfg );
- psz_urls = var_CreateGetString( p_filter, CFG_PREFIX "urls" );
+ /* Get the urls to parse: must be non empty */
+ psz_urls = var_CreateGetNonEmptyString( p_filter, CFG_PREFIX "urls" );
+ if( !psz_urls )
+ {
+ msg_Err( p_filter, "The list of urls must not be empty" );
+ free( p_sys );
+ return VLC_EGENERIC;
+ }
+
+ /* Fill the p_sys structure with the configuration */
p_sys->i_title = var_CreateGetInteger( p_filter, CFG_PREFIX "title" );
p_sys->i_cur_feed = 0;
p_sys->i_cur_item = p_sys->i_title == scroll_title ? -1 : 0;
p_sys->p_feeds = NULL;
p_sys->i_speed = var_CreateGetInteger( p_filter, CFG_PREFIX "speed" );
p_sys->i_length = var_CreateGetInteger( p_filter, CFG_PREFIX "length" );
- p_sys->i_ttl = __MAX( 0, var_CreateGetInteger( p_filter, CFG_PREFIX "ttl" ) );
p_sys->b_images = var_CreateGetBool( p_filter, CFG_PREFIX "images" );
+ i_ttl = __MAX( 0, var_CreateGetInteger( p_filter, CFG_PREFIX "ttl" ) );
+
p_sys->psz_marquee = malloc( p_sys->i_length + 1 );
if( p_sys->psz_marquee == NULL )
- goto error;
+ {
+ free( psz_urls );
+ free( p_sys );
+ return VLC_ENOMEM;
+ }
p_sys->psz_marquee[p_sys->i_length] = '\0';
p_sys->p_style = text_style_New();
p_sys->p_style->i_font_color = var_CreateGetInteger( p_filter, CFG_PREFIX "color" );
p_sys->p_style->i_font_size = var_CreateGetInteger( p_filter, CFG_PREFIX "size" );
- if( p_sys->b_images == true && p_sys->p_style->i_font_size == -1 )
+ if( p_sys->b_images && p_sys->p_style->i_font_size == -1 )
{
msg_Warn( p_filter, "rss-size wasn't specified. Feed images will thus be displayed without being resized" );
}
/* Parse the urls */
if( ParseUrls( p_filter, psz_urls ) )
- {
- free( psz_urls );
goto error;
- }
- free( psz_urls );
- if( FetchRSS( p_filter ) )
- {
- msg_Err( p_filter, "failed while fetching RSS ... too bad" );
- text_style_Delete( p_sys->p_style );
- i_ret = VLC_EGENERIC;
- goto error;
- }
- p_sys->t_last_update = time( NULL );
+ /* Misc init */
+ vlc_mutex_init( &p_sys->lock );
+ p_filter->pf_sub_source = Filter;
+ p_sys->last_date = (mtime_t)0;
+ p_sys->b_fetched = false;
- if( p_sys->i_feeds == 0 )
+ /* Create and arm the timer */
+ if( vlc_timer_create( &p_sys->timer, Fetch, p_filter ) )
{
- text_style_Delete( p_sys->p_style );
- i_ret = VLC_EGENERIC;
+ vlc_mutex_destroy( &p_sys->lock );
goto error;
}
- for( i_feed=0; i_feed < p_sys->i_feeds; i_feed ++ )
- {
- if( p_sys->p_feeds[i_feed].i_items == 0 )
- {
- DestroyFilter( p_this );
- return VLC_EGENERIC;
- }
- }
-
- /* Misc init */
- vlc_mutex_init( &p_sys->lock );
- p_filter->pf_sub_filter = Filter;
- p_sys->last_date = (mtime_t)0;
+ vlc_timer_schedule( p_sys->timer, false, 1,
+ (mtime_t)(i_ttl)*1000000 );
+ free( psz_urls );
return VLC_SUCCESS;
error:
+ if( p_sys->p_style )
+ text_style_Delete( p_sys->p_style );
free( p_sys->psz_marquee );
+ free( psz_urls );
free( p_sys );
- return i_ret;
+ return VLC_ENOMEM;
}
/*****************************************************************************
* DestroyFilter: destroy RSS video filter
filter_t *p_filter = (filter_t *)p_this;
filter_sys_t *p_sys = p_filter->p_sys;
+ vlc_timer_destroy( p_sys->timer );
vlc_mutex_destroy( &p_sys->lock );
+
text_style_Delete( p_sys->p_style );
free( p_sys->psz_marquee );
- FreeRSS( p_filter );
+ FreeRSS( p_sys->p_feeds, p_sys->i_feeds );
free( p_sys );
}
vlc_mutex_lock( &p_sys->lock );
+ /* Check if the feeds have been fetched and that we have some feeds */
+ /* TODO: check that we have items for each feeds */
+ if( !p_sys->b_fetched && p_sys->i_feeds > 0 )
+ {
+ vlc_mutex_unlock( &p_sys->lock );
+ return NULL;
+ }
+
if( p_sys->last_date
+ ( p_sys->i_cur_char == 0 && p_sys->i_cur_item == ( p_sys->i_title == scroll_title ? -1 : 0 ) ? 5 : 1 )
/* ( ... ? 5 : 1 ) means "wait 5 times more for the 1st char" */
return NULL;
}
- /* Do we need to update the feeds ? */
- if( p_sys->i_ttl
- && time( NULL ) > p_sys->t_last_update + (time_t)p_sys->i_ttl )
- {
- msg_Dbg( p_filter, "Forcing update of all the RSS feeds" );
- if( FetchRSS( p_filter ) )
- {
- msg_Err( p_filter, "Failed while fetching RSS ... too bad" );
- vlc_mutex_unlock( &p_sys->lock );
- return NULL; /* FIXME : we most likely messed up all the data,
- * so we might need to do something about it */
- }
- p_sys->t_last_update = time( NULL );
- }
-
p_sys->last_date = date;
p_sys->i_cur_char++;
if( p_sys->i_cur_item == -1 ? p_sys->p_feeds[p_sys->i_cur_feed].psz_title[p_sys->i_cur_char] == 0 : p_sys->p_feeds[p_sys->i_cur_feed].p_items[p_sys->i_cur_item].psz_title[p_sys->i_cur_char] == 0 )
/* where to locate the string: */
if( p_sys->i_pos < 0 )
{ /* set to an absolute xy */
- p_spu->p_region->i_align = OSD_ALIGN_LEFT | OSD_ALIGN_TOP;
+ p_spu->p_region->i_align = SUBPICTURE_ALIGN_LEFT | SUBPICTURE_ALIGN_TOP;
p_spu->b_absolute = true;
}
else
p_spu->p_region->i_align = p_sys->i_pos;
p_spu->b_absolute = false;
}
+ p_spu->p_region->i_x = p_sys->i_xoff;
+ p_spu->p_region->i_y = p_sys->i_yoff;
p_spu->p_region->p_style = text_style_Duplicate( p_sys->p_style );
memset( &fmt_out, 0, sizeof(video_format_t) );
fmt_out.i_chroma = VLC_CODEC_YUVA;
- fmt_out.i_aspect = VOUT_ASPECT_FACTOR;
fmt_out.i_sar_num = fmt_out.i_sar_den = 1;
fmt_out.i_width =
fmt_out.i_visible_width = p_pic->p[Y_PLANE].i_visible_pitch;
}
else
{
- p_region->i_x = p_sys->i_xoff;
- p_region->i_y = p_sys->i_yoff;
+ p_region->i_x = p_spu->p_region->i_x;
+ p_region->i_y = p_spu->p_region->i_y;
/* FIXME the copy is probably not needed anymore */
picture_Copy( p_region->p_picture, p_pic );
p_spu->p_region->p_next = p_region;
- }
- /* Offset text to display right next to the image */
- p_spu->p_region->i_x = p_pic->p[Y_PLANE].i_visible_pitch;
+ /* Offset text to display right next to the image */
+ p_spu->p_region->i_x += fmt_out.i_visible_width;
+ }
}
vlc_mutex_unlock( &p_sys->lock );
/****************************************************************************
- * Parse url list, psz_urls must be non empty
+ * Parse url list, psz_urls must be non empty (TODO: check it !)
***************************************************************************/
static int ParseUrls( filter_t *p_filter, char *psz_urls )
{
rss_feed_t *p_feed )
{
VLC_UNUSED(p_filter);
+ const char *node;
char *psz_eltname = NULL;
bool b_is_item = false;
bool b_is_image = false;
int i_item = 0;
+ int type;
- while( xml_ReaderRead( p_xml_reader ) == 1 )
+ while( (type = xml_ReaderNextNode( p_xml_reader, &node )) > 0 )
{
- switch( xml_ReaderNodeType( p_xml_reader ) )
+ switch( type )
{
- // Error
- case -1:
- goto end;
-
case XML_READER_STARTELEM:
- free( psz_eltname );
- psz_eltname = xml_ReaderName( p_xml_reader );
- if( !psz_eltname )
- goto end;
-
#ifdef RSS_DEBUG
- msg_Dbg( p_filter, "element name: %s", psz_eltname );
+ msg_Dbg( p_filter, "element <%s>", node );
#endif
+ psz_eltname = strdup( node );
+ if( unlikely(!psz_eltname) )
+ goto end;
+
/* rss or atom */
- if( !strcmp( psz_eltname, "item" ) || !strcmp( psz_eltname, "entry" ) )
+ if( !strcmp( node, "item" ) || !strcmp( node, "entry" ) )
{
b_is_item = true;
p_feed->i_items++;
- p_feed->p_items = realloc( p_feed->p_items,
- p_feed->i_items * sizeof( rss_item_t ) );
+ p_feed->p_items = xrealloc( p_feed->p_items,
+ p_feed->i_items * sizeof( rss_item_t ) );
p_feed->p_items[p_feed->i_items-1].psz_title = NULL;
p_feed->p_items[p_feed->i_items-1].psz_description = NULL;
p_feed->p_items[p_feed->i_items-1].psz_link = NULL;
}
/* rss */
- else if( !strcmp( psz_eltname, "image" ) )
+ else if( !strcmp( node, "image" ) )
{
b_is_image = true;
}
/* atom */
- else if( !strcmp( psz_eltname, "link" ) )
+ else if( !strcmp( node, "link" ) )
{
+ const char *name, *value;
char *psz_href = NULL;
char *psz_rel = NULL;
- while( xml_ReaderNextAttr( p_xml_reader ) == VLC_SUCCESS )
+
+ while( (name = xml_ReaderNextAttr( p_xml_reader, &value )) != NULL )
{
- char *psz_name = xml_ReaderName( p_xml_reader );
- char *psz_value = xml_ReaderValue( p_xml_reader );
- if( !strcmp( psz_name, "rel" ) )
+ if( !strcmp( name, "rel" ) )
{
free( psz_rel );
- psz_rel = psz_value;
+ psz_rel = strdup( value );
}
- else if( !strcmp( psz_name, "href" ) )
+ else if( !strcmp( name, "href" ) )
{
free( psz_href );
- psz_href = psz_value;
- }
- else
- {
- free( psz_value );
+ psz_href = strdup( value );
}
- free( psz_name );
}
/* "rel" and "href" must be defined */
break;
case XML_READER_ENDELEM:
- free( psz_eltname );
- psz_eltname = xml_ReaderName( p_xml_reader );
- if( !psz_eltname )
- goto end;
-
+ FREENULL( psz_eltname );
#ifdef RSS_DEBUG
- msg_Dbg( p_filter, "element end : %s", psz_eltname );
+ msg_Dbg( p_filter, "element end </%s>", node );
#endif
/* rss or atom */
- if( !strcmp( psz_eltname, "item" ) || !strcmp( psz_eltname, "entry" ) )
+ if( !strcmp( node, "item" ) || !strcmp( node, "entry" ) )
{
b_is_item = false;
i_item++;
}
/* rss */
- else if( !strcmp( psz_eltname, "image" ) )
+ else if( !strcmp( node, "image" ) )
{
b_is_image = false;
}
- FREENULL( psz_eltname );
break;
case XML_READER_TEXT:
{
if( !psz_eltname )
break;
- char *psz_eltvalue = xml_ReaderValue( p_xml_reader );
- if( !psz_eltvalue )
- goto end;
- char *psz_clean = removeWhiteChars( psz_eltvalue );
- free( psz_eltvalue );
- psz_eltvalue = psz_clean;
+ char *psz_eltvalue = removeWhiteChars( node );
#ifdef RSS_DEBUG
- msg_Dbg( p_filter, " text : <%s>", psz_eltvalue );
+ msg_Dbg( p_filter, " text : \"%s\"", psz_eltvalue );
#endif
/* Is it an item ? */
if( b_is_item )
/****************************************************************************
* FetchRSS (or Atom) feeds
***************************************************************************/
-static int FetchRSS( filter_t *p_filter )
+static rss_feed_t* FetchRSS( filter_t *p_filter )
{
filter_sys_t *p_sys = p_filter->p_sys;
stream_t *p_stream;
xml_t *p_xml;
xml_reader_t *p_xml_reader;
+ int i_feed;
+
+ /* These data are not modified after the creation of the module so we don't
+ need to hold the lock */
+ int i_feeds = p_sys->i_feeds;
+ bool b_images = p_sys->b_images;
+
+ /* Allocate a new structure */
+ rss_feed_t *p_feeds = malloc( i_feeds * sizeof( rss_feed_t ) );
+ if( !p_feeds )
+ return NULL;
p_xml = xml_Create( p_filter );
if( !p_xml )
{
msg_Err( p_filter, "Failed to open XML parser" );
- return 1;
+ free( p_feeds );
+ return NULL;
}
/* Fetch all feeds and parse them */
- for( int i_feed = 0; i_feed < p_sys->i_feeds; i_feed++ )
+ for( i_feed = 0; i_feed < i_feeds; i_feed++ )
{
- rss_feed_t *p_feed = p_sys->p_feeds+i_feed;
-
- /* Free the ressources */
- FREENULL( p_feed->psz_title );
- FREENULL( p_feed->psz_description );
- FREENULL( p_feed->psz_link );
- FREENULL( p_feed->psz_image );
- if( p_feed->p_pic )
- {
- picture_Release( p_feed->p_pic );
- p_feed->p_pic = NULL;
- }
- for( int i = 0; i < p_feed->i_items; i++ )
- {
- rss_item_t *p_item = p_feed->p_items + i;
- free( p_item->psz_title );
- free( p_item->psz_link );
- free( p_item->psz_description );
- }
+ rss_feed_t *p_feed = p_feeds + i_feed;
+ rss_feed_t *p_old_feed = p_sys->p_feeds + i_feed;
+
+ /* Initialize the structure */
+ p_feed->psz_title = NULL;
+ p_feed->psz_description = NULL;
+ p_feed->psz_link = NULL;
+ p_feed->psz_image = NULL;
+ p_feed->p_pic = NULL;
p_feed->i_items = 0;
- FREENULL( p_feed->p_items );
+ p_feed->p_items = NULL;
+
+ p_feed->psz_url = strdup( p_old_feed->psz_url );
/* Fetch the feed */
msg_Dbg( p_filter, "opening %s RSS/Atom feed ...", p_feed->psz_url );
goto error;
/* If we have a image: load it if requiere */
- if( p_sys->b_images == true
- && p_feed->psz_image && !p_feed->p_pic )
+ if( b_images && p_feed->psz_image && !p_feed->p_pic )
{
p_feed->p_pic = LoadImage( p_filter, p_feed->psz_image );
}
msg_Dbg( p_filter, "done with %s RSS/Atom feed", p_feed->psz_url );
- xml_ReaderDelete( p_xml, p_xml_reader );
+ xml_ReaderDelete( p_xml_reader );
stream_Delete( p_stream );
}
xml_Delete( p_xml );
- return 0;
+ return p_feeds;
error:
-
+ FreeRSS( p_feeds, i_feed + 1 );
if( p_xml_reader )
- xml_ReaderDelete( p_xml, p_xml_reader );
+ xml_ReaderDelete( p_xml_reader );
if( p_stream )
stream_Delete( p_stream );
if( p_xml )
xml_Delete( p_xml );
- return 1;
+ return NULL;
}
/****************************************************************************
* FreeRSS
***************************************************************************/
-static void FreeRSS( filter_t *p_filter)
+static void FreeRSS( rss_feed_t *p_feeds, int i_feeds )
{
- filter_sys_t *p_sys = p_filter->p_sys;
-
- for( int i_feed = 0; i_feed < p_sys->i_feeds; i_feed++ )
+ for( int i_feed = 0; i_feed < i_feeds; i_feed++ )
{
- rss_feed_t *p_feed = p_sys->p_feeds+i_feed;
+ rss_feed_t *p_feed = p_feeds+i_feed;
for( int i_item = 0; i_item < p_feed->i_items; i_item++ )
{
rss_item_t *p_item = p_feed->p_items+i_item;
picture_Release( p_feed->p_pic );
free( p_feed->psz_url );
}
- free( p_sys->p_feeds );
- p_sys->i_feeds = 0;
+ free( p_feeds );
+}
+
+static void Fetch( void *p_data )
+{
+ filter_t *p_filter = p_data;
+ filter_sys_t *p_sys = p_filter->p_sys;
+
+ msg_Dbg( p_filter, "Updating the rss feeds" );
+ rss_feed_t *p_feeds = FetchRSS( p_filter );
+ if( !p_feeds )
+ {
+ msg_Err( p_filter, "Unable to fetch the feeds" );
+ return;
+ }
+
+ rss_feed_t *p_old_feeds = p_sys->p_feeds;
+
+ vlc_mutex_lock( &p_sys->lock );
+ /* Update the feeds */
+ p_sys->p_feeds = p_feeds;
+ p_sys->b_fetched = true;
+ /* Set all current info to the original values */
+ p_sys->i_cur_feed = 0;
+ p_sys->i_cur_item = p_sys->i_title == scroll_title ? -1 : 0;
+ p_sys->i_cur_char = 0;
+ vlc_mutex_unlock( &p_sys->lock );
+
+ if( p_old_feeds )
+ FreeRSS( p_old_feeds, p_sys->i_feeds );
}