X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=src%2Fui%2Fgui%2Fpsppire-dialog-action-rank.c;h=c0454d10f3b86a1855bad7faad9493a227dda7c8;hb=033449d465fb54526d3984d55772e6a38e7d922c;hp=16adb9f707b2ffeb390eb98b9ef8538776a2eada;hpb=258cbd9cdf386687122b0854274923acafc786d4;p=pspp

diff --git a/src/ui/gui/psppire-dialog-action-rank.c b/src/ui/gui/psppire-dialog-action-rank.c
index 16adb9f707..c0454d10f3 100644
--- a/src/ui/gui/psppire-dialog-action-rank.c
+++ b/src/ui/gui/psppire-dialog-action-rank.c
@@ -53,7 +53,7 @@ generate_syntax (const PsppireDialogAction *act)
 		   gtk_toggle_button_get_active (rd->ascending_togglebutton)
 		   ?'A':'D');
 
-  if (  gtk_tree_model_get_iter_first (gs, &notused) )
+  if (gtk_tree_model_get_iter_first (gs, &notused))
     {
       g_string_append (str, "\n\tBY ");
 
@@ -94,24 +94,24 @@ generate_syntax (const PsppireDialogAction *act)
     {
       g_string_append (str, "\n\t/FRACTION=");
 
-      if ( gtk_toggle_button_get_active (rd->blom))
+      if (gtk_toggle_button_get_active (rd->blom))
 	g_string_append (str, "BLOM");
-      else if ( gtk_toggle_button_get_active (rd->tukey))
+      else if (gtk_toggle_button_get_active (rd->tukey))
 	g_string_append (str, "TUKEY");
-      else if ( gtk_toggle_button_get_active (rd->rankit))
+      else if (gtk_toggle_button_get_active (rd->rankit))
 	g_string_append (str, "RANKIT");
-      else if ( gtk_toggle_button_get_active (rd->vw))
+      else if (gtk_toggle_button_get_active (rd->vw))
 	g_string_append (str, "VW");
     }
 
   g_string_append (str, "\n\t/TIES=");
-  if ( gtk_toggle_button_get_active (rd->mean))
+  if (gtk_toggle_button_get_active (rd->mean))
     g_string_append (str, "MEAN");
-  else if ( gtk_toggle_button_get_active (rd->low))
+  else if (gtk_toggle_button_get_active (rd->low))
     g_string_append (str, "LOW");
-  else if ( gtk_toggle_button_get_active (rd->high))
+  else if (gtk_toggle_button_get_active (rd->high))
     g_string_append (str, "HIGH");
-  else if ( gtk_toggle_button_get_active (rd->condense))
+  else if (gtk_toggle_button_get_active (rd->condense))
     g_string_append (str, "CONDENSE");
 
 
@@ -155,7 +155,7 @@ types_dialog_reset (PsppireDialogActionRank *rd)
 {
   gint i;
 
-  for (i = 0 ; i < n_RANK_FUNCS ; ++i )
+  for (i = 0 ; i < n_RANK_FUNCS ; ++i)
     gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (rd->func_button [i]),
 				  FALSE);
 
@@ -206,20 +206,13 @@ set_sensitivity (PsppireDialogActionRank *dar)
   gtk_widget_set_sensitive (dar->formula_box, sens);
 }
 
-static void
-psppire_dialog_action_rank_activate (PsppireDialogAction *a)
+static GtkBuilder *
+psppire_dialog_action_rank_activate (PsppireDialogAction *a, GVariant *param)
 {
   PsppireDialogAction *pda = PSPPIRE_DIALOG_ACTION (a);
   PsppireDialogActionRank *act = PSPPIRE_DIALOG_ACTION_RANK (a);
 
-  GHashTable *thing = psppire_dialog_action_get_hash_table (pda);
-  GtkBuilder *xml = g_hash_table_lookup (thing, a);
-  if (!xml)
-    {
-      xml = builder_new ("rank.ui");
-      g_hash_table_insert (thing, a, xml);
-    }
-
+  GtkBuilder *xml = builder_new ("rank.ui");
   GtkWidget *types_button = get_widget_assert (xml, "button1");
   GtkWidget *ties_button = get_widget_assert (xml, "button2");
 
@@ -298,12 +291,13 @@ psppire_dialog_action_rank_activate (PsppireDialogAction *a)
   psppire_dialog_action_set_valid_predicate (pda, (void *) dialog_state_valid);
   psppire_dialog_action_set_refresh (pda, dialog_refresh);
 
+  return xml;
 }
 
 static void
 psppire_dialog_action_rank_class_init (PsppireDialogActionRankClass *class)
 {
-  psppire_dialog_action_set_activation (class, psppire_dialog_action_rank_activate);
+  PSPPIRE_DIALOG_ACTION_CLASS (class)->initial_activate = psppire_dialog_action_rank_activate;
 
   PSPPIRE_DIALOG_ACTION_CLASS (class)->generate_syntax = generate_syntax;
 }