Crahing bug fixed :)
[rodin/chimara.git] / libchimara / graphics.c
index 48b3a12dfd4d18550e61ca54855dd1838b9b16e0..898c9dc32fa5b4f40d3f76b8815aa73d775ce51b 100644 (file)
@@ -6,8 +6,10 @@
 
 extern GPrivate *glk_data_key;
 void on_size_prepared(GdkPixbufLoader *loader, gint width, gint height, struct image_info *info);
+void on_pixbuf_closed(GdkPixbufLoader *loader, gpointer data);
 
 static gboolean size_determined;
+static gboolean image_loaded;
 
 glui32
 glk_image_get_info(glui32 image, glui32 *width, glui32 *height)
@@ -20,8 +22,6 @@ glk_image_get_info(glui32 image, glui32 *width, glui32 *height)
        info->resource_number = image;
        guchar *buffer;
 
-       //printf("glk_image_get_info(%d)\n", image);
-
        /* Lookup the proper resource */
        blorb_error = giblorb_load_resource(glk_data->resource_map, giblorb_method_FilePos, &res, giblorb_ID_Pict, image);
        if(blorb_error != giblorb_err_None) {
@@ -61,11 +61,12 @@ glk_image_get_info(glui32 image, glui32 *width, glui32 *height)
        g_free(buffer);
 
        /* Determine the image dimensions */
-       if(!size_determined) {
-               WARNING("Cannot read image size: file data trimmed?");
-               g_free(info);
-               return FALSE;
+       g_mutex_lock(glk_data->resource_lock);
+       while(!size_determined) {
+               /* Wait for the PixbufLoader to finish reading the image size */
+               g_cond_wait(glk_data->resource_info_available, glk_data->resource_lock);
        }
+       g_mutex_unlock(glk_data->resource_lock);
 
        if(width != NULL)
                *width = info->width;
@@ -73,30 +74,27 @@ glk_image_get_info(glui32 image, glui32 *width, glui32 *height)
                *height =info->height;
        g_free(info);
 
-       //printf("size loaded: %d x %d\n", (int) *width, (int) *height);
-
        return TRUE;
 }
 
 void
 on_size_prepared(GdkPixbufLoader *loader, gint width, gint height, struct image_info *info)
 {
+       ChimaraGlkPrivate *glk_data = g_private_get(glk_data_key);
+
+       g_mutex_lock(glk_data->resource_lock);
        info->width = width;
        info->height = height;
        size_determined = TRUE;
-}
-
-void
-on_graphics_size_allocate(GtkWidget *widget, GtkAllocation *allocation, winid_t *win)
-{
-       printf("size allocated: %dx%d\n", allocation->width, allocation->height);
+       g_cond_broadcast(glk_data->resource_info_available);
+       g_mutex_unlock(glk_data->resource_lock);
 }
 
 glui32
-glk_image_draw(winid_t win, glui32 image, glsi32 val1, glsi32 val2) {
+glk_image_draw(winid_t win, glui32 image, glsi32 val1, glsi32 val2)
+{
        VALID_WINDOW(win, return FALSE);
        g_return_val_if_fail(win->type == wintype_Graphics, FALSE);
-       printf("Drawing image %d\n", (int)image);
 
        ChimaraGlkPrivate *glk_data = g_private_get(glk_data_key);
        giblorb_result_t res;
@@ -107,8 +105,6 @@ glk_image_draw(winid_t win, glui32 image, glsi32 val1, glsi32 val2) {
        guchar *buffer;
        GdkPixmap *canvas;
 
-       //printf("glk_image_get_info(%d)\n", image);
-
        /* Lookup the proper resource */
        blorb_error = giblorb_load_resource(glk_data->resource_map, giblorb_method_FilePos, &res, giblorb_ID_Pict, image);
        if(blorb_error != giblorb_err_None) {
@@ -118,10 +114,12 @@ glk_image_draw(winid_t win, glui32 image, glsi32 val1, glsi32 val2) {
 
        /* Load the resource */
        GdkPixbufLoader *loader = gdk_pixbuf_loader_new();
+       g_signal_connect( loader, "closed", G_CALLBACK(on_pixbuf_closed), NULL ); 
        glk_stream_set_position(glk_data->resource_file, res.data.startpos, seekmode_Start);
        buffer = g_malloc( BUFFER_SIZE * sizeof(guchar) );
 
        guint32 total_read = 0;
+       image_loaded = FALSE;
        while(total_read < res.length) {
                guint32 num_read = glk_get_buffer_stream(glk_data->resource_file, (char *) buffer, BUFFER_SIZE);
 
@@ -135,34 +133,70 @@ glk_image_draw(winid_t win, glui32 image, glsi32 val1, glsi32 val2) {
 
                total_read += num_read;
        }
+
+       if( !gdk_pixbuf_loader_close(loader, &pixbuf_error) ) {
+               WARNING_S("Cannot read image", pixbuf_error->message);
+               giblorb_unload_chunk(glk_data->resource_map, image);
+               g_free(buffer);
+               return FALSE;
+       }
+
+       if(!image_loaded) {
+               /* Wait for the PixbufLoader to finish loading the image */
+               g_mutex_lock(glk_data->resource_lock);
+               while(!image_loaded) {
+                       g_cond_wait(glk_data->resource_loaded, glk_data->resource_lock);
+               }
+               g_mutex_unlock(glk_data->resource_lock);
+       }
+
        giblorb_unload_chunk(glk_data->resource_map, image);
        g_free(buffer);
+       
+       gdk_threads_enter();
 
        gtk_image_get_pixmap( GTK_IMAGE(win->widget), &canvas, NULL );
        if(canvas == NULL) {
                WARNING("Could not get pixmap");
-               gdk_pixbuf_loader_close(loader, &pixbuf_error);
                return FALSE;
        }
 
        GdkPixbuf *pixbuf = gdk_pixbuf_loader_get_pixbuf(loader);
        if(pixbuf == NULL) {
                WARNING("Could not read image");
-               gdk_pixbuf_loader_close(loader, &pixbuf_error);
                return FALSE;
        }
 
        gdk_draw_pixbuf( GDK_DRAWABLE(canvas), NULL, pixbuf, 0, 0, val1, val2, -1, -1, GDK_RGB_DITHER_NONE, 0, 0 );
-       gdk_pixbuf_loader_close(loader, &pixbuf_error);
 
        /* Update the screen */
        gtk_widget_queue_draw(win->widget);
+
+       gdk_threads_leave();
+
        return TRUE;
 }
 
+void
+on_pixbuf_closed(GdkPixbufLoader *loader, gpointer data)
+{
+       gdk_threads_enter();
+
+       ChimaraGlkPrivate *glk_data = g_private_get(glk_data_key);
+
+       g_mutex_lock(glk_data->resource_lock);
+       image_loaded = TRUE;
+       g_cond_broadcast(glk_data->resource_loaded);
+       g_mutex_unlock(glk_data->resource_lock);
+
+       gdk_threads_leave();
+}
+
+
 glui32
 glk_image_draw_scaled(winid_t win, glui32 image, glsi32 val1, glsi32 val2, glui32 width, glui32 height)
 {
+       glk_image_draw(win, image, val1, val2);
        return TRUE;
 }
 
@@ -177,19 +211,58 @@ glk_window_fill_rect(winid_t win, glui32 color, glsi32 left, glsi32 top, glui32
        VALID_WINDOW(win, return);
        g_return_if_fail(win->type == wintype_Graphics);
 
+
+       gdk_threads_enter();
+
        GdkPixmap *map;
        gtk_image_get_pixmap( GTK_IMAGE(win->widget), &map, NULL );
        gdk_draw_rectangle( GDK_DRAWABLE(map), win->widget->style->white_gc, TRUE, left, top, width, height);
        gtk_widget_queue_draw(win->widget);
+
+       gdk_threads_leave();
 }
 
 void
 glk_window_erase_rect(winid_t win, glsi32 left, glsi32 top, glui32 width, glui32 height)
 {
-       printf("erasing rect: %d %d %d %d\n", left, top, width, height);
        glk_window_fill_rect(win, win->background_color, left, top, width, height);
 }
 
 void glk_window_flow_break(winid_t win)
 {
 }
+
+/*** Called when the graphics window is resized. Resize the backing pixmap if necessary ***/
+void
+on_graphics_size_allocate(GtkWidget *widget, GtkAllocation *allocation, winid_t win)
+{ 
+       GdkPixmap *oldmap;
+       gtk_image_get_pixmap( GTK_IMAGE(widget), &oldmap, NULL );
+       gint oldwidth = 0;
+       gint oldheight = 0;
+       /* Determine whether a pixmap exists with the correct size */
+       gboolean needs_resize = FALSE;
+       if(oldmap == NULL)
+               needs_resize = TRUE;
+       else {
+               gdk_drawable_get_size( GDK_DRAWABLE(oldmap), &oldwidth, &oldheight );
+               if(oldwidth != allocation->width || oldheight != allocation->height)
+                       needs_resize = TRUE;
+       }
+
+       if(needs_resize) {
+               /* Create a new pixmap */
+               GdkPixmap *newmap = gdk_pixmap_new(widget->window, allocation->width, allocation->height, -1);
+               gdk_draw_rectangle( GDK_DRAWABLE(newmap), widget->style->white_gc, TRUE, 0, 0, allocation->width, allocation->height);
+
+               /* Copy the contents of the old pixmap */
+               if(oldmap != NULL)
+                       gdk_draw_drawable( GDK_DRAWABLE(newmap), widget->style->white_gc, GDK_DRAWABLE(oldmap), 0, 0, 0, 0, oldwidth, oldheight);
+               
+               /* Use the new pixmap */
+               gtk_image_set_from_pixmap( GTK_IMAGE(widget), newmap, NULL );
+               g_object_unref(newmap);
+       }
+}
+