diff options
author | Lionel Landwerlin <lionel.g.landwerlin@linux.intel.com> | 2011-01-18 15:47:08 +0000 |
---|---|---|
committer | Iago Toral Quiroga <itoral@igalia.com> | 2011-01-19 08:31:21 +0100 |
commit | 1eb8ba7c5c4fee34e1b0e4eecf34f971f0a5d9dd (patch) | |
tree | e6c10367392b9462d7efe349604047b6dfc56e6d /src | |
parent | e50d532a65b751c00700ad81690df17410ae92b0 (diff) | |
download | grilo-plugins-1eb8ba7c5c4fee34e1b0e4eecf34f971f0a5d9dd.tar.gz |
plugins: sync Grilo config usage with rework in core
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@linux.intel.com>
Signed-off-by: Iago Toral Quiroga <itoral@igalia.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/apple-trailers/grl-apple-trailers.c | 20 | ||||
-rw-r--r-- | src/filesystem/grl-filesystem.c | 10 | ||||
-rw-r--r-- | src/flickr/grl-flickr.c | 13 | ||||
-rw-r--r-- | src/vimeo/grl-vimeo.c | 18 | ||||
-rw-r--r-- | src/youtube/grl-youtube.c | 9 |
5 files changed, 47 insertions, 23 deletions
diff --git a/src/apple-trailers/grl-apple-trailers.c b/src/apple-trailers/grl-apple-trailers.c index 28764ad..5054853 100644 --- a/src/apple-trailers/grl-apple-trailers.c +++ b/src/apple-trailers/grl-apple-trailers.c @@ -114,21 +114,27 @@ grl_apple_trailers_plugin_init (GrlPluginRegistry *registry, for (; configs; configs = g_list_next (configs)) { GrlConfig *config; - const gchar *definition, *poster_size; + gchar *definition, *poster_size; config = GRL_CONFIG (configs->data); definition = grl_config_get_string (config, "definition"); - if (definition && *definition != '\0') { - if (g_str_equal (definition, "hd")) { - hd = TRUE; + if (definition) { + if (*definition != '\0') { + if (g_str_equal (definition, "hd")) { + hd = TRUE; + } } + g_free (definition); } poster_size = grl_config_get_string (config, "poster-size"); - if (poster_size && *poster_size != '\0') { - if (g_str_equal (poster_size, "xlarge")) { - xlarge = TRUE; + if (poster_size) { + if (*poster_size != '\0') { + if (g_str_equal (poster_size, "xlarge")) { + xlarge = TRUE; + } } + g_free (poster_size); } } diff --git a/src/filesystem/grl-filesystem.c b/src/filesystem/grl-filesystem.c index 3933a6a..475620d 100644 --- a/src/filesystem/grl-filesystem.c +++ b/src/filesystem/grl-filesystem.c @@ -152,16 +152,14 @@ grl_filesystem_plugin_init (GrlPluginRegistry *registry, NULL); for (; configs; configs = g_list_next (configs)) { - const gchar *path; - const GValue *value; + gchar *path; config = GRL_CONFIG (configs->data); path = grl_config_get_string (config, GRILO_CONF_CHOSEN_PATH); if (path) { - chosen_paths = g_list_append (chosen_paths, g_strdup (path)); + chosen_paths = g_list_append (chosen_paths, path); } - value = grl_config_get (config, GRILO_CONF_MAX_SEARCH_DEPTH); - if (value && G_VALUE_HOLDS_INT (value)) { - max_search_depth = (guint)g_value_get_int (value); + if (grl_config_has_param (config, GRILO_CONF_MAX_SEARCH_DEPTH)) { + max_search_depth = (guint)grl_config_get_int (config, GRILO_CONF_MAX_SEARCH_DEPTH); } } source->priv->chosen_paths = chosen_paths; diff --git a/src/flickr/grl-flickr.c b/src/flickr/grl-flickr.c index cbd08b6..0869e4f 100644 --- a/src/flickr/grl-flickr.c +++ b/src/flickr/grl-flickr.c @@ -117,9 +117,9 @@ grl_flickr_plugin_init (GrlPluginRegistry *registry, const GrlPluginInfo *plugin, GList *configs) { - const gchar *flickr_key; - const gchar *flickr_secret; - const gchar *flickr_token; + gchar *flickr_key; + gchar *flickr_secret; + gchar *flickr_token; GrlConfig *config; gboolean public_source_created = FALSE; @@ -157,6 +157,13 @@ grl_flickr_plugin_init (GrlPluginRegistry *registry, NULL); } + if (flickr_key != NULL) + g_free (flickr_key); + if (flickr_token != NULL) + g_free (flickr_token); + if (flickr_secret != NULL) + g_free (flickr_secret); + configs = g_list_next (configs); } diff --git a/src/vimeo/grl-vimeo.c b/src/vimeo/grl-vimeo.c index 2664232..f3c513a 100644 --- a/src/vimeo/grl-vimeo.c +++ b/src/vimeo/grl-vimeo.c @@ -87,10 +87,11 @@ grl_vimeo_plugin_init (GrlPluginRegistry *registry, const GrlPluginInfo *plugin, GList *configs) { - const gchar *vimeo_key; - const gchar *vimeo_secret; + gchar *vimeo_key; + gchar *vimeo_secret; GrlConfig *config; gint config_count; + gboolean init_result = FALSE; GrlVimeoSource *source; GRL_LOG_DOMAIN_INIT (vimeo_log_domain, "vimeo"); @@ -118,7 +119,7 @@ grl_vimeo_plugin_init (GrlPluginRegistry *registry, if (!vimeo_key || !vimeo_secret) { GRL_WARNING ("Required configuration keys not set up"); - return FALSE; + goto go_out; } source = grl_vimeo_source_new (); @@ -128,7 +129,16 @@ grl_vimeo_plugin_init (GrlPluginRegistry *registry, plugin, GRL_MEDIA_PLUGIN (source), NULL); - return TRUE; + init_result = TRUE; + + go_out: + + if (vimeo_key != NULL) + g_free (vimeo_key); + if (vimeo_secret != NULL) + g_free (vimeo_secret); + + return init_result; } GRL_PLUGIN_REGISTER (grl_vimeo_plugin_init, diff --git a/src/youtube/grl-youtube.c b/src/youtube/grl-youtube.c index 8b05047..ccf1a02 100644 --- a/src/youtube/grl-youtube.c +++ b/src/youtube/grl-youtube.c @@ -250,7 +250,7 @@ grl_youtube_plugin_init (GrlPluginRegistry *registry, const GrlPluginInfo *plugin, GList *configs) { - const gchar *api_key; + gchar *api_key; GrlConfig *config; gint config_count; @@ -287,6 +287,9 @@ grl_youtube_plugin_init (GrlPluginRegistry *registry, plugin, GRL_MEDIA_PLUGIN (source), NULL); + + g_free (api_key); + return TRUE; } @@ -655,7 +658,7 @@ build_media_from_entry (GrlMedia *content, } else if (iter->data == GRL_METADATA_KEY_SITE) { grl_media_set_site (media, gdata_youtube_video_get_player_uri (video)); } else if (iter->data == GRL_METADATA_KEY_EXTERNAL_URL) { - grl_media_set_external_url (media, + grl_media_set_external_url (media, gdata_youtube_video_get_player_uri (video)); } else if (iter->data == GRL_METADATA_KEY_RATING) { gdouble average; @@ -821,7 +824,7 @@ item_count_cb (GObject *object, GAsyncResult *result, CategoryCountCb *cc) GDataFeed *feed; GError *error = NULL; - + feed = gdata_service_query_finish (GDATA_SERVICE (cc->service), result, &error); if (error) { |