Avoid excess threads with g_mutex_trylock

[grits] / src / plugins / elev.c

diff --git a/src/plugins/elev.c b/src/plugins/elev.c
index 3a5c031685cba4904fc66c249549fb98e848a901..9279b08190ca9c6e1ec314077b2e4cc52d26da73 100644 (file)
--- a/src/plugins/elev.c
+++ b/src/plugins/elev.c
@@ -163,8 +163,8 @@ static guint _load_opengl(GdkPixbuf *pixbuf)
                        (alpha ? GL_RGBA : GL_RGB), GL_UNSIGNED_BYTE, pixels);
        glTexParameterf(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_NEAREST);
        glTexParameterf(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_NEAREST);
-       glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, GL_CLAMP_TO_BORDER);
-       glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, GL_CLAMP_TO_BORDER);
+       glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, GL_CLAMP);
+       glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, GL_CLAMP);
 
        g_debug("GisPluginElev: load_opengl %d", opengl);
        return opengl;
@@ -246,7 +246,8 @@ static void _free_tile(GisTile *tile, gpointer _elev)
 static gpointer _update_tiles(gpointer _elev)
 {
        GisPluginElev *elev = _elev;
-       g_mutex_lock(elev->mutex);
+       if (!g_mutex_trylock(elev->mutex))
+               return NULL;
        gdouble lat, lon, elevation;
        gis_viewer_get_location(elev->viewer, &lat, &lon, &elevation);
        gis_tile_update(elev->tiles,
@@ -268,6 +269,13 @@ static void _on_location_changed(GisViewer *viewer,
        g_thread_create(_update_tiles, elev, FALSE, NULL);
 }
 
+static gpointer _threaded_init(GisPluginElev *elev)
+{
+       _load_tile(elev->tiles, elev);
+       _update_tiles(elev);
+       return NULL;
+}
+
 /***********
  * Methods *
  ***********/
@@ -286,8 +294,7 @@ GisPluginElev *gis_plugin_elev_new(GisViewer *viewer)
        elev->viewer = g_object_ref(viewer);
 
        /* Load initial tiles */
-       _load_tile(elev->tiles, elev);
-       g_thread_create(_update_tiles, elev, FALSE, NULL);
+       g_thread_create((GThreadFunc)_threaded_init, elev, FALSE, NULL);
 
        /* Connect signals */
        elev->sigid = g_signal_connect(elev->viewer, "location-changed",