aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--mail/ChangeLog13
-rw-r--r--mail/mail-display.c88
-rw-r--r--mail/mail-display.h1
3 files changed, 93 insertions, 9 deletions
diff --git a/mail/ChangeLog b/mail/ChangeLog
index 1a190ceaf9..ec9b1c4028 100644
--- a/mail/ChangeLog
+++ b/mail/ChangeLog
@@ -1,3 +1,16 @@
+2000-12-26 Iain Holmes <iain@helixcode.com>
+
+ * mail-display.c (mail_display_init): Initialise the thumbnail cache.
+ (mail_display_destroy): Free the cache.
+ (pixbuf_gen_idle): Check the cache for a pixbuf, add the pixbuf to the
+ cache if it's not there.
+
+2000-12-26 Iain Holmes <iain@helixcode.com>
+
+ * mail-summary.c (create_summary_view): Create a shared
+ BonoboEventSource object and use it for all the objects that
+ aggregate Bonobo::EventSource.
+
2000-12-27 Dan Winship <danw@helixcode.com>
* subscribe-dialog.c (setup_subscribe_folder): Use info->name, not
diff --git a/mail/mail-display.c b/mail/mail-display.c
index b7d4442a2a..41a5daad1a 100644
--- a/mail/mail-display.c
+++ b/mail/mail-display.c
@@ -40,11 +40,16 @@ static GtkObjectClass *mail_display_parent_class;
static void redisplay (MailDisplay *md, gboolean unscroll);
struct _PixbufLoader {
+ MailDisplay *md;
CamelDataWrapper *wrapper; /* The data */
CamelStream *mstream;
GdkPixbufLoader *loader;
+ GtkHTMLEmbedded *eb;
char *type; /* Type of data, in case the conversion fails */
+ char *cid; /* Strdupped on creation, but not freed until
+ the hashtable is destroyed */
GtkWidget *pixmap;
+ guint32 destroy_id;
};
/*----------------------------------------------------------------------*
@@ -427,8 +432,29 @@ pixbuf_gen_idle (struct _PixbufLoader *pbl)
char tmp[4096];
int len, width, height, ratio;
- /* Get a pixbuf from the wrapper */
+ /* Get the pixbuf from the cache */
+ mini = g_hash_table_lookup (pbl->md->thumbnail_cache, pbl->cid);
+ if (mini) {
+ width = gdk_pixbuf_get_width (mini);
+ height = gdk_pixbuf_get_height (mini);
+ bonobo_ui_toolbar_icon_set_pixbuf (
+ BONOBO_UI_TOOLBAR_ICON (pbl->pixmap), mini);
+ gtk_widget_set_usize (pbl->pixmap, width, height);
+
+ if (pbl->loader) {
+ gdk_pixbuf_loader_close (pbl->loader);
+ gtk_object_destroy (GTK_OBJECT (pbl->loader));
+ camel_object_unref (CAMEL_OBJECT (pbl->mstream));
+ }
+ gtk_signal_disconnect (GTK_OBJECT (pbl->eb), pbl->destroy_id);
+ g_free (pbl->type);
+ g_free (pbl);
+
+ return FALSE;
+ }
+
+ /* Not in cache, so get a pixbuf from the wrapper */
if (!GTK_IS_WIDGET (pbl->pixmap)) {
/* Widget has died */
if (pbl->mstream)
@@ -464,7 +490,7 @@ pixbuf_gen_idle (struct _PixbufLoader *pbl)
width = gdk_pixbuf_get_width (pixbuf);
height = gdk_pixbuf_get_height (pixbuf);
-
+
if (width >= height) {
if (width > 24) {
ratio = width / 24;
@@ -485,9 +511,13 @@ pixbuf_gen_idle (struct _PixbufLoader *pbl)
gdk_pixbuf_unref (pixbuf);
bonobo_ui_toolbar_icon_set_pixbuf (
BONOBO_UI_TOOLBAR_ICON (pbl->pixmap), mini);
- gdk_pixbuf_unref (mini);
+
+ /* Add the pixbuf to the cache */
+
+ g_hash_table_insert (pbl->md->thumbnail_cache, pbl->cid, mini);
gtk_widget_set_usize (pbl->pixmap, width, height);
+ gtk_signal_disconnect (GTK_OBJECT (pbl->eb), pbl->destroy_id);
if (pbl->loader) {
gdk_pixbuf_loader_close (pbl->loader);
gtk_object_destroy (GTK_OBJECT (pbl->loader));
@@ -498,6 +528,26 @@ pixbuf_gen_idle (struct _PixbufLoader *pbl)
return FALSE;
}
+/* Stop the idle function and free the pbl structure
+ as the widget that the pixbuf was to be rendered to
+ has died on us. */
+static void
+embeddable_destroy_cb (GtkObject *embeddable,
+ struct _PixbufLoader *pbl)
+{
+ g_idle_remove_by_data (pbl);
+ if (pbl->mstream)
+ camel_object_unref (CAMEL_OBJECT (pbl->mstream));
+
+ if (pbl->loader) {
+ gdk_pixbuf_loader_close (pbl->loader);
+ gtk_object_destroy (GTK_OBJECT (pbl->loader));
+ }
+
+ g_free (pbl->type);
+ g_free (pbl);
+};
+
static gboolean
on_object_requested (GtkHTML *html, GtkHTMLEmbedded *eb, gpointer data)
{
@@ -543,7 +593,14 @@ on_object_requested (GtkHTML *html, GtkHTMLEmbedded *eb, gpointer data)
camel_stream_reset (pbl->mstream);
}
pbl->type = g_strdup (eb->type);
+ pbl->cid = g_strdup (cid);
pbl->pixmap = bonobo_ui_toolbar_icon_new ();
+ pbl->eb = eb;
+ pbl->md = md;
+ pbl->destroy_id = gtk_signal_connect (GTK_OBJECT (eb),
+ "destroy",
+ embeddable_destroy_cb,
+ pbl);
g_idle_add_full (G_PRIORITY_LOW, (GSourceFunc)pixbuf_gen_idle,
pbl, NULL);
@@ -552,8 +609,6 @@ on_object_requested (GtkHTML *html, GtkHTMLEmbedded *eb, gpointer data)
gtk_widget_set_sensitive (GTK_WIDGET (ebox), TRUE);
gtk_widget_add_events (GTK_WIDGET (ebox),
GDK_BUTTON_PRESS_MASK);
- gtk_signal_connect (GTK_OBJECT (ebox), "button_press_event",
- GTK_SIGNAL_FUNC (pixmap_press), ebox);
gtk_object_set_data (GTK_OBJECT (ebox), "MailDisplay", md);
gtk_object_set_data (GTK_OBJECT (ebox), "CamelMimePart",
medium);
@@ -561,6 +616,9 @@ on_object_requested (GtkHTML *html, GtkHTMLEmbedded *eb, gpointer data)
g_strdup (eb->type),
(GDestroyNotify)g_free);
+ gtk_signal_connect (GTK_OBJECT (ebox), "button_press_event",
+ GTK_SIGNAL_FUNC (pixmap_press), ebox);
+
gtk_container_add (GTK_CONTAINER (ebox), pbl->pixmap);
gtk_widget_show_all (ebox);
gtk_container_add (GTK_CONTAINER (eb), ebox);
@@ -772,11 +830,10 @@ redisplay (MailDisplay *md, gboolean unscroll)
adj = e_scroll_frame_get_vadjustment (md->scroll);
oldv = adj->value;
}
-
- md->stream = gtk_html_begin (md->html);
+ md->stream = gtk_html_begin (GTK_HTML (md->html));
mail_html_write (md->html, md->stream, "%s%s", HTML_HEADER,
"<BODY TEXT=\"#000000\" BGCOLOR=\"#FFFFFF\">\n");
-
+
if (md->current_message) {
camel_object_ref (CAMEL_OBJECT (md->current_message));
if (mail_config_view_source ())
@@ -784,7 +841,7 @@ redisplay (MailDisplay *md, gboolean unscroll)
else
mail_format_mime_message (md->current_message, md);
}
-
+
mail_html_write (md->html, md->stream, "</BODY></HTML>\n");
gtk_html_end (md->html, md->stream, GTK_HTML_STREAM_OK);
md->stream = NULL;
@@ -859,6 +916,16 @@ mail_display_init (GtkObject *object)
/* various other initializations */
mail_display->current_message = NULL;
+ mail_display->thumbnail_cache = g_hash_table_new (g_str_hash, g_str_equal);
+}
+
+static void
+thumbnail_cache_free (gpointer key,
+ gpointer value,
+ gpointer user_data)
+{
+ g_free (key);
+ gdk_pixbuf_unref (value);
}
static void
@@ -866,6 +933,9 @@ mail_display_destroy (GtkObject *object)
{
MailDisplay *mail_display = MAIL_DISPLAY (object);
+ g_hash_table_foreach (mail_display->thumbnail_cache,
+ thumbnail_cache_free, NULL);
+ g_hash_table_destroy (mail_display->thumbnail_cache);
g_datalist_clear (mail_display->data);
g_free (mail_display->data);
diff --git a/mail/mail-display.h b/mail/mail-display.h
index c4eac5f145..c7c45c9bdf 100644
--- a/mail/mail-display.h
+++ b/mail/mail-display.h
@@ -30,6 +30,7 @@ struct _MailDisplay {
CamelMimeMessage *current_message;
GData **data;
+ GHashTable *thumbnail_cache;
};
typedef struct {