gpointer user_data)
{
MvsMInfoProviderService *self = MVS_MINFO_PROVIDER_SERVICE (user_data);
- GError *error = NULL;
GList *iter = NULL;
- gchar **object_paths= g_new0 (gchar*, g_list_length (list) + 1);
+ gchar **object_paths= g_new0 (gchar*, g_list_length (list) + 1);
gchar *movie_interface = NULL;
guint i = 0;
g_strdup (TMDB_MOVIE_INTERFACE):
g_strdup (WATC_MOVIE_INTERFACE);
+ /* TODO We left the movie var unused to keep a warning
+ * remembering that DBus objects aren't being freed
+ * when they aren't needed anymore */
for (iter = list; iter; iter = iter->next) {
if (MVS_IS_TMDB_MOVIE (iter->data)) {
MvsTmdbMovie *tmdb_movie = MVS_TMDB_MOVIE (iter->data);
* id="4bc91...e007304"/> */
for (i = 0; i < LAST_FIELD; i ++) {
- value = xmlGetProp (node, image_fields[i]);
- g_object_set (image, image_fields[i], value, NULL);
+ const gchar *image_field = mvs_tmdb_image_get_field (i);
- g_free (value);
+ value = xmlGetProp (node, image_field);
+ g_object_set (image, image_field, value, NULL);
+
+ g_free (value);
}
return image;
NULL,
NULL,
XML_PARSE_NOBLANKS | XML_PARSE_RECOVER);
- g_return_if_fail (document);
+ g_return_val_if_fail (document, NULL);
xmlXPathContextPtr context_ptr = xmlXPathNewContext (document);
gchar *id;
};
+static const gchar*
+image_fields[LAST_FIELD] = { "type", "url", "size", "id" };
+
static void
mvs_tmdb_image_get_property (GObject *object, guint property_id,
GValue *value, GParamSpec *pspec)
gboolean
mvs_tmdb_image_set_imagetype (MvsTmdbImage *self, const gchar *type)
{
- g_return_val_if_fail(MVS_TMDB_IMAGE (self), FALSE);
+ g_return_val_if_fail (MVS_TMDB_IMAGE (self), FALSE);
g_free (self->priv->type);
gboolean
mvs_tmdb_image_set_url (MvsTmdbImage *self, const gchar *url)
{
- g_return_val_if_fail(MVS_TMDB_IMAGE (self), FALSE);
+ g_return_val_if_fail (MVS_TMDB_IMAGE (self), FALSE);
g_free (self->priv->url);
gboolean
mvs_tmdb_image_set_size (MvsTmdbImage *self, const gchar *size)
{
- g_return_val_if_fail(MVS_TMDB_IMAGE (self), FALSE);
+ g_return_val_if_fail (MVS_TMDB_IMAGE (self), FALSE);
g_free (self->priv->size);
gboolean
mvs_tmdb_image_set_id (MvsTmdbImage *self, const gchar *id)
{
- g_return_val_if_fail(MVS_TMDB_IMAGE (self), FALSE);
+ g_return_val_if_fail (MVS_TMDB_IMAGE (self), FALSE);
g_free (self->priv->id);
gboolean
mvs_tmdb_image_set_width (MvsTmdbImage *self, gint width)
{
- g_return_val_if_fail(MVS_TMDB_IMAGE (self), FALSE);
+ g_return_val_if_fail (MVS_TMDB_IMAGE (self), FALSE);
self->priv->width = width;
gboolean
mvs_tmdb_image_set_height (MvsTmdbImage *self, gint height)
{
- g_return_val_if_fail(MVS_TMDB_IMAGE (self), FALSE);
+ g_return_val_if_fail (MVS_TMDB_IMAGE (self), FALSE);
self->priv->height = height;
g_free (value);
}
}
+
+const gchar*
+mvs_tmdb_image_get_field (int field)
+{
+ g_return_val_if_fail (field >= 0 && field < LAST_FIELD, NULL);
+
+ return image_fields[field];
+}
LAST_FIELD
};
-static const gchar*
-image_fields[LAST_FIELD] = { "type", "url", "size", "id" };
-
GType mvs_tmdb_image_get_type (void);
MvsTmdbImage* mvs_tmdb_image_new ();
void
mvs_tmdb_image_print (MvsTmdbImage *self);
+const gchar*
+mvs_tmdb_image_get_field (int field);
+
G_END_DECLS
#endif /* _MVS_TMDB_IMAGE */
g_free (self->priv->year);
self->priv->year = g_strdup (year);
+
+ return self->priv->year != NULL;
}
const gchar*
mvs_watc_movie_set_stingers (MvsWatcMovie *self, int stingers);
gint
-mvs_watch_movie_get_stingers (MvsWatcMovie *self);
+mvs_watc_movie_get_stingers (MvsWatcMovie *self);
gboolean
mvs_watc_movie_set_year (MvsWatcMovie *self, const gchar *year);
mvs_watc_movie_get_year (MvsWatcMovie *self);
void
-mvs_watch_movie_print (MvsWatcMovie *self);
+mvs_watc_movie_print (MvsWatcMovie *self);
G_END_DECLS