[Xfce4-commits] <parole:master> Enhanced audio cover image loading, scale proportionally, store at the largest used image size.
Sean Davis
noreply at xfce.org
Wed Jan 9 01:38:01 CET 2013
Updating branch refs/heads/master
to d96e68e3db30964ef883f9fda229a87361939b96 (commit)
from e781a38713d482249137d161b81a902452967d88 (commit)
commit d96e68e3db30964ef883f9fda229a87361939b96
Author: Sean Davis <smd.seandavis at gmail.com>
Date: Tue Jan 8 19:36:55 2013 -0500
Enhanced audio cover image loading, scale proportionally, store at the largest used image size.
src/gst/parole-gst.c | 171 ++++++++++++++++++++++++++++++--------------------
src/parole-player.c | 6 +-
2 files changed, 106 insertions(+), 71 deletions(-)
diff --git a/src/gst/parole-gst.c b/src/gst/parole-gst.c
index 5d08e64..885d5a0 100644
--- a/src/gst/parole-gst.c
+++ b/src/gst/parole-gst.c
@@ -68,7 +68,7 @@ static void parole_gst_change_state (ParoleGst *gst,
static void parole_gst_terminate_internal (ParoleGst *gst,
gboolean fade_sound);
-static gchar * parole_gst_tag_list_get_cover_external (ParoleGst *gst);
+static GdkPixbuf * parole_gst_tag_list_get_cover_external (ParoleGst *gst);
static GdkPixbuf * parole_gst_tag_list_get_cover (ParoleGst *gst, GstTagList *tag_list);
@@ -1099,9 +1099,11 @@ parole_gst_buffer_to_pixbuf (GstBuffer *buffer)
return pixbuf;
}
-gchar *
+GdkPixbuf *
parole_gst_tag_list_get_cover_external (ParoleGst *gst)
{
+ GdkPixbuf *pixbuf;
+ GError *err = NULL;
gchar *uri;
gchar *filename;
gchar *directory;
@@ -1152,96 +1154,131 @@ parole_gst_tag_list_get_cover_external (ParoleGst *gst)
g_free(directory);
g_free(file_dir);
g_free(lower);
- return cover_filename;
+
+ if (!cover_filename)
+ return NULL;
+
+ pixbuf = gdk_pixbuf_new_from_file(cover_filename, &err);
+ g_free(cover_filename);
+ if (err)
+ {
+ g_object_unref(pixbuf);
+ g_error_free(err);
+ return NULL;
+ }
+ return pixbuf;
}
-static const GValue *
-parole_gst_tag_list_get_cover_real (ParoleGst *gst, GstTagList *tag_list)
+GdkPixbuf *
+parole_gst_tag_list_get_cover_embedded (ParoleGst *gst, GstTagList *tag_list)
{
+ GdkPixbuf *pixbuf;
#if GST_CHECK_VERSION(1, 0, 0)
- GstPad *videopad;
- GstCaps *caps;
+ GstPad *videopad;
+ GstCaps *caps;
#endif
- const GValue *cover_value = NULL;
- guint i;
+ const GValue *cover_value = NULL;
+ guint i;
- for (i = 0; ; i++) {
- const GValue *value;
+ for (i = 0; ; i++) {
+ const GValue *value;
#if GST_CHECK_VERSION(1, 0, 0)
#else
- GstBuffer *buffer;
+ GstBuffer *buffer;
#endif
- GstStructure *caps_struct;
- int type;
+ GstStructure *caps_struct;
+ int type;
- value = gst_tag_list_get_value_index (tag_list,
- GST_TAG_IMAGE,
- i);
- if (value == NULL)
- break;
+ value = gst_tag_list_get_value_index ( tag_list,
+ GST_TAG_IMAGE,
+ i);
+ if (value == NULL)
+ break;
#if GST_CHECK_VERSION(1, 0, 0)
- g_signal_emit_by_name (gst->priv->playbin, "get-video-pad", i, &videopad);
- caps = gst_pad_query_caps(videopad, NULL);
- caps_struct = gst_caps_get_structure (caps, 0);
- gst_caps_unref(caps);
- g_object_unref (videopad);
+ g_signal_emit_by_name (gst->priv->playbin, "get-video-pad", i, &videopad);
+ caps = gst_pad_query_caps(videopad, NULL);
+ caps_struct = gst_caps_get_structure (caps, 0);
+ gst_caps_unref(caps);
+ g_object_unref (videopad);
#else
- buffer = gst_value_get_buffer (value);
- caps_struct = gst_caps_get_structure (buffer->caps, 0);
+ buffer = gst_value_get_buffer (value);
+ caps_struct = gst_caps_get_structure (buffer->caps, 0);
#endif
-
- gst_structure_get_enum (caps_struct,
- "image-type",
- GST_TYPE_TAG_IMAGE_TYPE,
- &type);
- if (type == GST_TAG_IMAGE_TYPE_UNDEFINED) {
- if (cover_value == NULL)
- cover_value = value;
- } else if (type == GST_TAG_IMAGE_TYPE_FRONT_COVER) {
- cover_value = value;
- break;
+
+ gst_structure_get_enum (caps_struct,
+ "image-type",
+ GST_TYPE_TAG_IMAGE_TYPE,
+ &type);
+ if (type == GST_TAG_IMAGE_TYPE_UNDEFINED) {
+ if (cover_value == NULL)
+ cover_value = value;
+ } else if (type == GST_TAG_IMAGE_TYPE_FRONT_COVER) {
+ cover_value = value;
+ break;
+ }
}
- }
- return cover_value;
+ if (!cover_value) {
+ cover_value = gst_tag_list_get_value_index (tag_list,
+ GST_TAG_PREVIEW_IMAGE,
+ 0);
+ }
+
+ if (cover_value) {
+ GstBuffer *buffer;
+ GdkPixbuf *pixbuf;
+
+ buffer = gst_value_get_buffer (cover_value);
+ pixbuf = parole_gst_buffer_to_pixbuf (buffer);
+ return pixbuf;
+ }
+ return NULL;
}
GdkPixbuf *
parole_gst_tag_list_get_cover (ParoleGst *gst, GstTagList *tag_list)
{
- gchar *cover_filename;
- const GValue *cover_value;
-
-
- g_return_val_if_fail (tag_list != NULL, FALSE);
-
- cover_filename = parole_gst_tag_list_get_cover_external(gst);
- if (cover_filename)
- {
- GdkPixbuf *pixbuf = gdk_pixbuf_new_from_file_at_size(cover_filename, 256, 256, NULL);
- if (pixbuf)
- return pixbuf;
- }
+ GdkPixbuf *pixbuf;
+ GdkPixbuf *scaled;
+ gint height, width;
+ gfloat multiplier;
- cover_value = parole_gst_tag_list_get_cover_real (gst, tag_list);
- /* Fallback to preview */
- if (!cover_value) {
- cover_value = gst_tag_list_get_value_index (tag_list,
- GST_TAG_PREVIEW_IMAGE,
- 0);
- }
+ g_return_val_if_fail (tag_list != NULL, FALSE);
- if (cover_value) {
- GstBuffer *buffer;
- GdkPixbuf *pixbuf;
+ pixbuf = parole_gst_tag_list_get_cover_external(gst);
+ if (!pixbuf)
+ {
+ pixbuf = parole_gst_tag_list_get_cover_embedded(gst, tag_list);
+ if (!pixbuf)
+ return NULL;
+ }
+
+ if (gdk_pixbuf_get_width(pixbuf) == gdk_pixbuf_get_height(pixbuf))
+ {
+ height = 256;
+ width = 256;
+ }
+ else if (gdk_pixbuf_get_width(pixbuf) < gdk_pixbuf_get_height(pixbuf))
+ {
+ multiplier = gdk_pixbuf_get_height(pixbuf)/256.0;
+ height = 256;
+ width = gdk_pixbuf_get_width(pixbuf) / multiplier;
+ }
+ else
+ {
+ multiplier = gdk_pixbuf_get_width(pixbuf)/256.0;
+ height = gdk_pixbuf_get_height(pixbuf)/multiplier;
+ width = 256;
+ }
- buffer = gst_value_get_buffer (cover_value);
- pixbuf = parole_gst_buffer_to_pixbuf (buffer);
- return pixbuf;
- }
+ scaled = gdk_pixbuf_scale_simple (pixbuf,
+ width,
+ height,
+ GDK_INTERP_HYPER);
- return NULL;
+ g_object_unref(pixbuf);
+ return scaled;
}
static void
diff --git a/src/parole-player.c b/src/parole-player.c
index 9f59e8f..de90e7c 100644
--- a/src/parole-player.c
+++ b/src/parole-player.c
@@ -1683,7 +1683,7 @@ parole_player_media_tag_cb (ParoleGst *gst, const ParoleStream *stream, ParolePl
gchar *album;
gchar *artist;
gchar *year;
- GdkPixbuf *image = NULL, *cover = NULL;
+ GdkPixbuf *image = NULL;
if ( player->priv->row )
{
@@ -1730,9 +1730,7 @@ parole_player_media_tag_cb (ParoleGst *gst, const ParoleStream *stream, ParolePl
image = parole_stream_get_image(G_OBJECT(stream));
if (image)
{
- cover = gdk_pixbuf_scale_simple(image, 256, 256, GDK_INTERP_BILINEAR);
- gtk_image_set_from_pixbuf(GTK_IMAGE(player->priv->audiobox_cover), cover);
- g_object_unref(cover);
+ gtk_image_set_from_pixbuf(GTK_IMAGE(player->priv->audiobox_cover), image);
g_object_unref(image);
}
else
More information about the Xfce4-commits
mailing list