# HG changeset patch
# User bsmith@81767d24-ef19-dc11-ae90-00e081727c95
# Date 1046326146 0
# Node ID 84471d53ab58dfc6f6b5c0b91ee98053ece4f501
# Parent  20becf7df38a530c5b22e3fb2db3a852990b47e1
Changes to dw_main_* so that the _dw_thread is set to -1 when the
message loop isn't running.  Also changed dw_icon_load_from_file() to
do more error checks and use unsigned long for returning.

diff -r 20becf7df38a -r 84471d53ab58 gtk/dw.c
--- a/gtk/dw.c	Tue Feb 25 20:38:12 2003 +0000
+++ b/gtk/dw.c	Thu Feb 27 06:09:06 2003 +0000
@@ -838,10 +838,11 @@
  */
 void dw_main(void)
 {
+	gdk_threads_enter();
 	_dw_thread = pthread_self();
 	_dw_thread_add(_dw_thread);
-	gdk_threads_enter();
 	gtk_main();
+	_dw_thread = (pthread_t)-1;
 	gdk_threads_leave();
 }
 
@@ -880,10 +881,11 @@
  */
 void dw_main_iteration(void)
 {
+	gdk_threads_enter();
 	_dw_thread = pthread_self();
 	_dw_thread_add(_dw_thread);
-	gdk_threads_enter();
 	gtk_main_iteration();
+	_dw_thread = (pthread_t)-1;
 	gdk_threads_leave();
 }
 
@@ -4152,13 +4154,14 @@
  */
 unsigned long API dw_icon_load_from_file(char *filename)
 {
-	int z, found = -1, _locked_by_me = FALSE;
+	int found = -1, _locked_by_me = FALSE;
 #if GTK_MAJOR_VERSION > 1
 	GdkPixbuf *pixbuf;
 #elif defined(USE_IMLIB)
 	GdkImlibImage *image;
 #endif
 	char *file = alloca(strlen(filename) + 5);
+	unsigned long z, ret = 0;
 
 	if (!file)
 		return 0;
@@ -4180,7 +4183,7 @@
 	{
 		if(!_PixmapArray[z].used)
 		{
-			found = z;
+			ret = found = z;
 			break;
 		}
 	}
@@ -4192,7 +4195,7 @@
 	{
 		DWPrivatePixmap *old = _PixmapArray;
 
-		found = _PixmapCount;
+		ret = found = _PixmapCount;
 		_PixmapCount++;
 
 		_PixmapArray = malloc(sizeof(DWPrivatePixmap) * _PixmapCount);
@@ -4208,26 +4211,38 @@
 #if GTK_MAJOR_VERSION > 1
 	pixbuf = gdk_pixbuf_new_from_file(file, NULL);
 
-	_PixmapArray[found].width = gdk_pixbuf_get_width(pixbuf);
-	_PixmapArray[found].height = gdk_pixbuf_get_height(pixbuf);
-
-	gdk_pixbuf_render_pixmap_and_mask(pixbuf, &_PixmapArray[found].pixmap, &_PixmapArray[found].mask, 1);
-	g_object_unref(pixbuf);
+	if(pixbuf)
+	{
+		_PixmapArray[found].width = gdk_pixbuf_get_width(pixbuf);
+		_PixmapArray[found].height = gdk_pixbuf_get_height(pixbuf);
+
+		gdk_pixbuf_render_pixmap_and_mask(pixbuf, &_PixmapArray[found].pixmap, &_PixmapArray[found].mask, 1);
+		g_object_unref(pixbuf);
+	}
 #elif defined(USE_IMLIB)
 	image = gdk_imlib_load_image(file);
 
-	_PixmapArray[found].width = image->rgb_width;
-	_PixmapArray[found].height = image->rgb_height;
-
-	gdk_imlib_render(image, image->rgb_width, image->rgb_height);
-	_PixmapArray[found].pixmap = gdk_imlib_copy_image(image);
-	_PixmapArray[found].mask = gdk_imlib_copy_mask(image);
-	gdk_imlib_destroy_image(image);
+	if(image)
+	{
+		_PixmapArray[found].width = image->rgb_width;
+		_PixmapArray[found].height = image->rgb_height;
+
+		gdk_imlib_render(image, image->rgb_width, image->rgb_height);
+		_PixmapArray[found].pixmap = gdk_imlib_copy_image(image);
+		_PixmapArray[found].mask = gdk_imlib_copy_mask(image);
+		gdk_imlib_destroy_image(image);
+	}
 #else
 	_PixmapArray[found].pixmap = gdk_pixmap_create_from_xpm(handle->window, &_PixmapArray[found].mask, &_colors[DW_CLR_PALEGRAY], file);
 #endif
 	DW_MUTEX_UNLOCK;
-	return found | (1 << 31);
+	if(!_PixmapArray[found].pixmap || !_PixmapArray[found].mask)
+	{
+		_PixmapArray[found].used = 0;
+		_PixmapArray[found].pixmap = _PixmapArray[found].mask = NULL;
+		return 0;
+	}
+	return ret | (1 << 31);
 }
 
 /*