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 5ea94ea00fb9749403d0264d6ad63b573395f51f..9279b08190ca9c6e1ec314077b2e4cc52d26da73 100644 (file)
--- a/src/plugins/elev.c
+++ b/src/plugins/elev.c
@@ -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",