]> git.street.me.uk Git - andy/viking.git/blob - src/viklayer.c
ChangeLog entry
[andy/viking.git] / src / viklayer.c
1 /*
2  * viking -- GPS Data and Topo Analyzer, Explorer, and Manager
3  *
4  * Copyright (C) 2003-2005, Evan Battaglia <gtoevan@gmx.net>
5  *
6  * This program is free software; you can redistribute it and/or modify
7  * it under the terms of the GNU General Public License as published by
8  * the Free Software Foundation; either version 2 of the License, or
9  * (at your option) any later version.
10  *
11  * This program is distributed in the hope that it will be useful,
12  * but WITHOUT ANY WARRANTY; without even the implied warranty of
13  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
14  * GNU General Public License for more details.
15  *
16  * You should have received a copy of the GNU General Public License
17  * along with this program; if not, write to the Free Software
18  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
19  *
20  */
21
22 #include "viking.h"
23 #include "vikradiogroup.h"
24 #include <string.h>
25
26 /* functions common to all layers. */
27 /* TODO longone: rename interface free -> finalize */
28
29 extern VikLayerInterface vik_aggregate_layer_interface;
30 extern VikLayerInterface vik_trw_layer_interface;
31 extern VikLayerInterface vik_maps_layer_interface;
32 extern VikLayerInterface vik_coord_layer_interface;
33 extern VikLayerInterface vik_georef_layer_interface;
34 extern VikLayerInterface vik_gps_layer_interface;
35
36 enum {
37   VL_UPDATE_SIGNAL,
38   VL_LAST_SIGNAL
39 };
40 static guint layer_signals[VL_LAST_SIGNAL] = { 0 };
41
42 static GObjectClass *parent_class;
43
44 static void layer_class_init ( VikLayerClass *klass );
45 static void layer_init ( VikLayer *vl );
46 static void layer_finalize ( VikLayer *vl );
47 static gboolean layer_properties_factory ( VikLayer *vl, gpointer vp );
48
49
50 /* TODO longone: rename vik_layer_init -> set_type */
51
52 GType vik_layer_get_type ()
53 {
54   static GType vl_type = 0;
55
56   if (!vl_type)
57   {
58     static const GTypeInfo vl_info =
59     {
60       sizeof (VikLayerClass),
61       NULL, /* base_init */
62       NULL, /* base_finalize */
63       (GClassInitFunc) layer_class_init, /* class init */
64       NULL, /* class_finalize */
65       NULL, /* class_data */
66       sizeof (VikLayer),
67       0,
68       (GInstanceInitFunc) layer_init /* instance init */
69     };
70     vl_type = g_type_register_static ( G_TYPE_OBJECT, "VikLayer", &vl_info, 0 );
71   }
72
73   return vl_type;
74 }
75
76 static void layer_class_init (VikLayerClass *klass)
77 {
78   GObjectClass *object_class;
79
80   object_class = G_OBJECT_CLASS (klass);
81
82   object_class->finalize = (GObjectFinalizeFunc) layer_finalize;
83
84   parent_class = g_type_class_peek_parent (klass);
85
86   layer_signals[VL_UPDATE_SIGNAL] = g_signal_new ( "update", G_TYPE_FROM_CLASS (klass),
87       G_SIGNAL_RUN_FIRST | G_SIGNAL_ACTION, G_STRUCT_OFFSET (VikLayerClass, update), NULL, NULL, 
88       g_cclosure_marshal_VOID__VOID, G_TYPE_NONE, 0);
89 }
90
91 void vik_layer_emit_update ( VikLayer *vl )
92 {
93   if ( vl->visible )
94     g_signal_emit ( G_OBJECT(vl), layer_signals[VL_UPDATE_SIGNAL], 0 );
95 }
96
97 static VikLayerInterface *vik_layer_interfaces[VIK_LAYER_NUM_TYPES] = {
98   &vik_aggregate_layer_interface,
99   &vik_trw_layer_interface,
100   &vik_coord_layer_interface,
101   &vik_georef_layer_interface,
102   &vik_gps_layer_interface,
103   &vik_maps_layer_interface,
104 };
105
106 VikLayerInterface *vik_layer_get_interface ( gint type )
107 {
108   g_assert ( type < VIK_LAYER_NUM_TYPES );
109   return vik_layer_interfaces[type];
110 }
111
112 static void layer_init ( VikLayer *vl )
113 {
114   vl->visible = TRUE;
115   vl->name = NULL;
116   vl->realized = FALSE;
117 }
118
119 void vik_layer_init ( VikLayer *vl, gint type )
120 {
121   vl->type = type;
122 }
123
124 /* frees old name */
125 void vik_layer_rename ( VikLayer *l, const gchar *new_name )
126 {
127   g_assert ( l != NULL);
128   if ( l->name )
129     g_free ( l->name );
130   l->name = g_strdup ( new_name );
131 }
132
133 void vik_layer_rename_no_copy ( VikLayer *l, gchar *new_name )
134 {
135   g_assert ( l != NULL);
136   if ( l->name )
137     g_free ( l->name );
138   l->name = new_name;
139 }
140
141 VikLayer *vik_layer_create ( gint type, gpointer vp, GtkWindow *w, gboolean interactive )
142 {
143   VikLayer *new_layer = NULL;
144   g_assert ( type < VIK_LAYER_NUM_TYPES );
145
146   new_layer = vik_layer_interfaces[type]->create ( vp );
147
148   g_assert ( new_layer != NULL );
149
150   if ( interactive )
151   {
152     if ( vik_layer_properties ( new_layer, vp ) )
153       vik_layer_rename ( VIK_LAYER(new_layer), vik_layer_interfaces[type]->name );
154     else
155     {
156       g_object_unref ( G_OBJECT(new_layer) ); /* cancel that */
157       new_layer = NULL;
158     }
159   }
160   return new_layer;
161 }
162
163 /* returns TRUE if OK was pressed */
164 gboolean vik_layer_properties ( VikLayer *layer, gpointer vp )
165 {
166   if ( vik_layer_interfaces[layer->type]->properties )
167     return vik_layer_interfaces[layer->type]->properties ( layer, vp );
168   return layer_properties_factory ( layer, vp );
169 }
170
171 void vik_layer_draw ( VikLayer *l, gpointer data )
172 {
173   if ( l->visible )
174     if ( vik_layer_interfaces[l->type]->draw )
175       vik_layer_interfaces[l->type]->draw ( l, data );
176 }
177
178 void vik_layer_change_coord_mode ( VikLayer *l, VikCoordMode mode )
179 {
180   if ( vik_layer_interfaces[l->type]->change_coord_mode )
181     vik_layer_interfaces[l->type]->change_coord_mode ( l, mode );
182 }
183
184 VikLayer *vik_layer_copy ( VikLayer *vl, gpointer vp )
185 {
186   if ( vik_layer_interfaces[vl->type]->copy )
187   {
188     VikLayer *rv = vik_layer_interfaces[vl->type]->copy ( vl, vp );
189     if ( rv )
190     {
191       vik_layer_rename ( rv, vl->name );
192       rv->visible = vl->visible;
193     }
194     return rv;
195   }
196   else
197     return NULL;
198 }
199
200 typedef struct {
201   gint layer_type;
202   gint len;
203   guint8 data[0];
204 } header_t;
205
206 void vik_layer_marshall ( VikLayer *vl, guint8 **data, gint *len )
207 {
208   header_t *header;
209   if ( vl && vik_layer_interfaces[vl->type]->marshall ) {
210     vik_layer_interfaces[vl->type]->marshall ( vl, data, len );
211     if (*data) {
212       header = g_malloc(*len + sizeof(*header));
213       header->layer_type = vl->type;
214       header->len = *len;
215       memcpy(header->data, *data, *len);
216       g_free(*data);
217       *data = (guint8 *)header;
218       *len = *len + sizeof(*header);
219     }
220   } else {
221     *data = NULL;
222   }
223 }
224
225 void vik_layer_marshall_params ( VikLayer *vl, guint8 **data, gint *datalen )
226 {
227   VikLayerParam *params = vik_layer_get_interface(vl->type)->params;
228   VikLayerFuncGetParam get_param = vik_layer_get_interface(vl->type)->get_param;
229   GByteArray* b = g_byte_array_new ();
230   gint len;
231
232 #define vlm_append(obj, sz)     \
233   len = (sz);                   \
234   g_byte_array_append ( b, (guint8 *)&len, sizeof(len) );       \
235   g_byte_array_append ( b, (guint8 *)(obj), len );
236
237   vlm_append(vl->name, strlen(vl->name));
238
239   if ( params && get_param )
240   {
241     VikLayerParamData d;
242     guint16 i, params_count = vik_layer_get_interface(vl->type)->params_count;
243     for ( i = 0; i < params_count; i++ )
244     {
245       d = get_param(vl, i);
246       switch ( params[i].type )
247       {
248       case VIK_LAYER_PARAM_STRING: 
249         vlm_append(d.s, strlen(d.s));
250         break;
251       default:
252         vlm_append(&d, sizeof(d));
253         break;
254       }
255     }
256   }
257   
258   *data = b->data;
259   *datalen = b->len;
260   g_byte_array_free ( b, FALSE );
261
262 #undef vlm_append
263 }
264
265 void vik_layer_unmarshall_params ( VikLayer *vl, guint8 *data, gint datalen, VikViewport *vvp )
266 {
267   VikLayerParam *params = vik_layer_get_interface(vl->type)->params;
268   VikLayerFuncSetParam set_param = vik_layer_get_interface(vl->type)->set_param;
269   gchar *s;
270   guint8 *b = (guint8 *)data;
271   
272 #define vlm_size (*(gint *)b)
273 #define vlm_read(obj)                           \
274   memcpy((obj), b+sizeof(gint), vlm_size);      \
275   b += sizeof(gint) + vlm_size;
276   
277   s = g_malloc(vlm_size + 1);
278   s[vlm_size]=0;
279   vlm_read(s);
280   
281   vik_layer_rename(vl, s);
282   
283   g_free(s);
284
285   if ( params && set_param )
286   {
287     VikLayerParamData d;
288     guint16 i, params_count = vik_layer_get_interface(vl->type)->params_count;
289     for ( i = 0; i < params_count; i++ )
290     {
291       switch ( params[i].type )
292       {
293       case VIK_LAYER_PARAM_STRING: 
294         s = g_malloc(vlm_size + 1);
295         s[vlm_size]=0;
296         vlm_read(s);
297         d.s = s;
298         set_param(vl, i, d, vvp);
299         g_free(s);
300         break;
301       default:
302         vlm_read(&d);
303         set_param(vl, i, d, vvp);
304         break;
305       }
306     }
307   }
308 }
309
310 VikLayer *vik_layer_unmarshall ( guint8 *data, gint len, VikViewport *vvp )
311 {
312   header_t *header;
313
314   header = (header_t *)data;
315   
316   if ( vik_layer_interfaces[header->layer_type]->unmarshall ) {
317     return vik_layer_interfaces[header->layer_type]->unmarshall ( header->data, header->len, vvp );
318   } else {
319     return NULL;
320   }
321 }
322
323 static void layer_finalize ( VikLayer *vl )
324 {
325   g_assert ( vl != NULL );
326   if ( vik_layer_interfaces[vl->type]->free )
327     vik_layer_interfaces[vl->type]->free ( vl );
328   if ( vl->name )
329     g_free ( vl->name );
330   G_OBJECT_CLASS(parent_class)->finalize(G_OBJECT(vl));
331 }
332
333 /* sublayer switching */
334 gboolean vik_layer_sublayer_toggle_visible ( VikLayer *l, gint subtype, gpointer sublayer )
335 {
336   if ( vik_layer_interfaces[l->type]->sublayer_toggle_visible )
337     return vik_layer_interfaces[l->type]->sublayer_toggle_visible ( l, subtype, sublayer );
338   return TRUE; /* if unknown, will always be visible */
339 }
340
341 void vik_layer_realize ( VikLayer *l, VikTreeview *vt, GtkTreeIter *layer_iter )
342 {
343   l->vt = vt;
344   l->iter = *layer_iter;
345   l->realized = TRUE;
346   if ( vik_layer_interfaces[l->type]->realize )
347     vik_layer_interfaces[l->type]->realize ( l, vt, layer_iter );
348 }
349
350 void vik_layer_set_menu_items_selection(VikLayer *l, guint16 selection)
351 {
352   if ( vik_layer_interfaces[l->type]->set_menu_selection )
353     vik_layer_interfaces[l->type]->set_menu_selection ( l, selection );
354 }
355
356 guint16 vik_layer_get_menu_items_selection(VikLayer *l)
357 {
358   if ( vik_layer_interfaces[l->type]->get_menu_selection )
359     return(vik_layer_interfaces[l->type]->get_menu_selection (l));
360   else
361     return(vik_layer_interfaces[l->type]->menu_items_selection);
362 }
363
364 void vik_layer_add_menu_items ( VikLayer *l, GtkMenu *menu, gpointer vlp )
365 {
366   if ( vik_layer_interfaces[l->type]->add_menu_items )
367     vik_layer_interfaces[l->type]->add_menu_items ( l, menu, vlp );
368 }
369
370 gboolean vik_layer_sublayer_add_menu_items ( VikLayer *l, GtkMenu *menu, gpointer vlp, gint subtype, gpointer sublayer, GtkTreeIter *iter )
371 {
372   if ( vik_layer_interfaces[l->type]->sublayer_add_menu_items )
373     return vik_layer_interfaces[l->type]->sublayer_add_menu_items ( l, menu, vlp, subtype, sublayer, iter );
374   return FALSE;
375 }
376
377
378 const gchar *vik_layer_sublayer_rename_request ( VikLayer *l, const gchar *newname, gpointer vlp, gint subtype, gpointer sublayer, GtkTreeIter *iter )
379 {
380   if ( vik_layer_interfaces[l->type]->sublayer_rename_request )
381     return vik_layer_interfaces[l->type]->sublayer_rename_request ( l, newname, vlp, subtype, sublayer, iter );
382   return NULL;
383 }
384
385 GdkPixbuf *vik_layer_load_icon ( gint type )
386 {
387   g_assert ( type < VIK_LAYER_NUM_TYPES );
388   if ( vik_layer_interfaces[type]->icon )
389     return gdk_pixbuf_from_pixdata ( vik_layer_interfaces[type]->icon, FALSE, NULL );
390   return NULL;
391 }
392
393 gboolean vik_layer_set_param ( VikLayer *layer, guint16 id, VikLayerParamData data, gpointer vp )
394 {
395   if ( vik_layer_interfaces[layer->type]->set_param )
396     return vik_layer_interfaces[layer->type]->set_param ( layer, id, data, vp );
397   return FALSE;
398 }
399
400 void vik_layer_post_read ( VikLayer *layer, gpointer vp )
401 {
402   if ( vik_layer_interfaces[layer->type]->post_read )
403     vik_layer_interfaces[layer->type]->post_read ( layer, vp );
404 }
405
406 static GtkWidget *properties_widget_new_widget ( VikLayerParam *param, VikLayerParamData data )
407 {
408   GtkWidget *rv = NULL;
409   switch ( param->widget_type )
410   {
411     case VIK_LAYER_WIDGET_COLOR:
412       if ( param->type == VIK_LAYER_PARAM_COLOR )
413         rv = gtk_color_button_new_with_color ( &(data.c) );
414       break;
415     case VIK_LAYER_WIDGET_CHECKBUTTON:
416       if ( param->type == VIK_LAYER_PARAM_BOOLEAN )
417       {
418         //rv = gtk_check_button_new_with_label ( //param->title );
419         rv = gtk_check_button_new ();
420         if ( data.b )
421           gtk_toggle_button_set_active ( GTK_TOGGLE_BUTTON(rv), TRUE );
422       }
423       break;
424     case VIK_LAYER_WIDGET_COMBOBOX:
425       if ( param->type == VIK_LAYER_PARAM_UINT && param->widget_data )
426       {
427         gchar **pstr = param->widget_data;
428         rv = gtk_combo_box_new_text ();
429         while ( *pstr )
430           gtk_combo_box_append_text ( GTK_COMBO_BOX ( rv ), *(pstr++) );
431         if ( param->extra_widget_data ) /* map of alternate uint values for options */
432         {
433           int i;
434           for ( i = 0; ((const char **)param->widget_data)[i]; i++ )
435             if ( ((guint *)param->extra_widget_data)[i] == data.u )
436             {
437               gtk_combo_box_set_active ( GTK_COMBO_BOX(rv), i );
438               break;
439             }
440         }
441         gtk_combo_box_set_active ( GTK_COMBO_BOX ( rv ), data.u );
442       }
443       break;
444     case VIK_LAYER_WIDGET_RADIOGROUP:
445       /* widget_data and extra_widget_data are GList */
446       if ( param->type == VIK_LAYER_PARAM_UINT && param->widget_data )
447       {
448         rv = vik_radio_group_new ( param->widget_data );
449         if ( param->extra_widget_data ) /* map of alternate uint values for options */
450         {
451           int i;
452           int nb_elem = g_list_length(param->widget_data);
453           for ( i = 0; i < nb_elem; i++ )
454             if ( g_list_nth_data(param->extra_widget_data, i) == data.u )
455             {
456               vik_radio_group_set_selected ( VIK_RADIO_GROUP(rv), i );
457               break;
458             }
459         }
460         else if ( data.u ) /* zero is already default */
461           vik_radio_group_set_selected ( VIK_RADIO_GROUP(rv), data.u );
462       }
463       break;
464     case VIK_LAYER_WIDGET_SPINBUTTON:
465       if ( (param->type == VIK_LAYER_PARAM_DOUBLE || param->type == VIK_LAYER_PARAM_UINT
466            || param->type == VIK_LAYER_PARAM_INT)  && param->widget_data )
467       {
468         gdouble init_val = (param->type == VIK_LAYER_PARAM_DOUBLE) ? data.d : (param->type == VIK_LAYER_PARAM_UINT ? data.u : data.i);
469         VikLayerParamScale *scale = (VikLayerParamScale *) param->widget_data;
470         rv = gtk_spin_button_new ( GTK_ADJUSTMENT(gtk_adjustment_new( init_val, scale->min, scale->max, scale->step, scale->step, scale->step )), scale->step, scale->digits );
471       }
472     break;
473     case VIK_LAYER_WIDGET_ENTRY:
474       if ( param->type == VIK_LAYER_PARAM_STRING )
475       {
476         rv = gtk_entry_new ();
477         gtk_entry_set_text ( GTK_ENTRY(rv), data.s );
478       }
479       break;
480     case VIK_LAYER_WIDGET_FILEENTRY:
481       if ( param->type == VIK_LAYER_PARAM_STRING )
482       {
483         rv = vik_file_entry_new ();
484         vik_file_entry_set_filename ( VIK_FILE_ENTRY(rv), data.s );
485       }
486       break;
487     case VIK_LAYER_WIDGET_HSCALE:
488       if ( (param->type == VIK_LAYER_PARAM_DOUBLE || param->type == VIK_LAYER_PARAM_UINT
489            || param->type == VIK_LAYER_PARAM_INT)  && param->widget_data )
490       {
491         gdouble init_val = (param->type == VIK_LAYER_PARAM_DOUBLE) ? data.d : (param->type == VIK_LAYER_PARAM_UINT ? data.u : data.i);
492         VikLayerParamScale *scale = (VikLayerParamScale *) param->widget_data;
493         rv = gtk_hscale_new_with_range ( scale->min, scale->max, scale->step );
494         gtk_scale_set_digits ( GTK_SCALE(rv), scale->digits );
495         gtk_range_set_value ( GTK_RANGE(rv), init_val );
496       }
497   }
498   return rv;
499 }
500
501 static VikLayerParamData properties_widget_get_value ( GtkWidget *widget, VikLayerParam *param )
502 {
503   VikLayerParamData rv;
504   switch ( param->widget_type )
505   {
506     case VIK_LAYER_WIDGET_COLOR:
507       gtk_color_button_get_color ( GTK_COLOR_BUTTON(widget), &(rv.c) );
508       break;
509     case VIK_LAYER_WIDGET_CHECKBUTTON:
510       rv.b = gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(widget));
511       break;
512     case VIK_LAYER_WIDGET_COMBOBOX:
513       rv.i = gtk_combo_box_get_active ( GTK_COMBO_BOX(widget) );
514       if ( rv.i == -1 ) rv.i = 0;
515       rv.u = rv.i;
516       if ( param->extra_widget_data )
517         rv.u = ((guint *)param->extra_widget_data)[rv.u];
518       break;
519     case VIK_LAYER_WIDGET_RADIOGROUP:
520       rv.u = vik_radio_group_get_selected(VIK_RADIO_GROUP(widget));
521       if ( param->extra_widget_data )
522         rv.u = (guint32)g_list_nth_data(param->extra_widget_data, rv.u);
523       break;
524     case VIK_LAYER_WIDGET_SPINBUTTON:
525       if ( param->type == VIK_LAYER_PARAM_UINT )
526         rv.u = gtk_spin_button_get_value_as_int ( GTK_SPIN_BUTTON(widget) );
527       else if ( param->type == VIK_LAYER_PARAM_INT )
528         rv.i = gtk_spin_button_get_value_as_int ( GTK_SPIN_BUTTON(widget) );
529       else
530         rv.d = gtk_spin_button_get_value ( GTK_SPIN_BUTTON(widget) );
531       break;
532     case VIK_LAYER_WIDGET_ENTRY:
533       rv.s = gtk_entry_get_text ( GTK_ENTRY(widget) );
534       break;
535     case VIK_LAYER_WIDGET_FILEENTRY:
536       rv.s = vik_file_entry_get_filename ( VIK_FILE_ENTRY(widget) );
537       break;
538     case VIK_LAYER_WIDGET_HSCALE:
539       if ( param->type == VIK_LAYER_PARAM_UINT )
540         rv.u = (guint32) gtk_range_get_value ( GTK_RANGE(widget) );
541       else if ( param->type == VIK_LAYER_PARAM_INT )
542         rv.i = (gint32) gtk_range_get_value ( GTK_RANGE(widget) );
543       else
544         rv.d = gtk_range_get_value ( GTK_RANGE(widget) );
545       break;
546   }
547   return rv;
548 }
549
550 /* false if cancel, true if OK */
551 /* some would claim this wasn't written to be human-readable. */
552 static gboolean layer_properties_factory ( VikLayer *vl, gpointer vp )
553 {
554   VikLayerParam *params = vik_layer_interfaces[vl->type]->params;
555   guint16 params_count = vik_layer_interfaces[vl->type]->params_count;
556   guint16 i, j, widget_count = 0;
557   gboolean must_redraw = FALSE;
558
559   if ( ! params )
560     return TRUE; /* no params == no options, so all is good */
561
562   for ( i = 0; i < params_count; i++ )
563     if ( params[i].group != VIK_LAYER_NOT_IN_PROPERTIES )
564       widget_count++;
565
566   if ( widget_count == 0)
567     return FALSE;
568   else
569   {
570     /* create widgets and titles; place in table */
571     GtkWidget *dialog = gtk_dialog_new_with_buttons ( "Layer Properties",
572                             VIK_GTK_WINDOW_FROM_WIDGET(vp),
573                             GTK_DIALOG_MODAL | GTK_DIALOG_DESTROY_WITH_PARENT,
574                             GTK_STOCK_CANCEL, GTK_RESPONSE_REJECT,
575                             GTK_STOCK_OK, GTK_RESPONSE_ACCEPT, NULL );
576     gint resp;
577
578     gchar **groups = vik_layer_interfaces[vl->type]->params_groups;
579     guint8 groups_count = vik_layer_interfaces[vl->type]->params_groups_count;
580
581     GtkWidget *table = NULL;
582     GtkWidget **tables = NULL; /* for more than one group */
583
584     GtkWidget *notebook = NULL;
585     GtkWidget **widgets = g_malloc ( sizeof(GtkWidget *) * widget_count );
586
587     if ( groups && groups_count )
588     {
589       guint8 current_group;
590       guint16 tab_widget_count;
591       notebook = gtk_notebook_new ();
592       gtk_box_pack_start (GTK_BOX(GTK_DIALOG(dialog)->vbox), notebook, FALSE, FALSE, 0);
593       tables = g_malloc ( sizeof(GtkWidget *) * groups_count );
594       for ( current_group = 0; current_group < groups_count; current_group++ )
595       {
596         tab_widget_count = 0;
597         for ( j = 0; j < params_count; j ++ )
598           if ( params[j].group == current_group )
599             tab_widget_count++;
600
601         if ( tab_widget_count )
602         {
603           tables[current_group] = gtk_table_new ( tab_widget_count, 1, FALSE );
604           gtk_notebook_append_page ( GTK_NOTEBOOK(notebook), tables[current_group], gtk_label_new(groups[current_group]) );
605         }
606       }
607     }
608     else
609     {
610       table = gtk_table_new( widget_count, 1, FALSE );
611       gtk_box_pack_start (GTK_BOX(GTK_DIALOG(dialog)->vbox), table, FALSE, FALSE, 0);
612     }
613
614     for ( i = 0, j = 0; i < params_count; i++ )
615     {
616       if ( params[i].group != VIK_LAYER_NOT_IN_PROPERTIES )
617       {
618         if ( tables )
619           table = tables[MAX(0, params[i].group)]; /* round up NOT_IN_GROUP, that's not reasonable here */
620
621         widgets[j] = properties_widget_new_widget ( &(params[i]),
622                          vik_layer_interfaces[vl->type]->get_param ( vl, i ) );
623
624         g_assert ( widgets[j] != NULL );
625
626         gtk_table_attach ( GTK_TABLE(table), gtk_label_new(params[i].title), 0, 1, j, j+1, 0, 0, 0, 0 );
627         gtk_table_attach ( GTK_TABLE(table), widgets[j], 1, 2, j, j+1, GTK_EXPAND | GTK_FILL, 0, 2, 2 );
628         j++;
629       }
630     }
631
632     gtk_widget_show_all ( dialog );
633
634     resp = gtk_dialog_run (GTK_DIALOG (dialog));
635     if ( resp == GTK_RESPONSE_ACCEPT )
636     {
637       for ( i = 0, j = 0; i < params_count; i++ )
638       {
639         if ( params[i].group != VIK_LAYER_NOT_IN_PROPERTIES )
640         {
641           if ( vik_layer_interfaces[vl->type]->set_param ( vl, i,
642               properties_widget_get_value ( widgets[j], &(params[i]) ), vp ) )
643             must_redraw = TRUE;
644           j++;
645         }
646       }
647       vik_layer_post_read ( vl, vp ); /* update any gc's */
648
649       gtk_widget_destroy ( dialog ); /* hide before redrawing. */
650       g_free ( widgets );
651
652       if ( must_redraw )
653         vik_layer_emit_update ( vl ); /* if this is a new layer, it won't redraw twice because no on'es listening to this signal. */
654       return TRUE; /* user clicked OK */
655     }
656
657     if ( tables )
658       g_free ( tables );
659     gtk_widget_destroy ( dialog );
660     g_free ( widgets );
661     return FALSE;
662   }
663 }