variable: Simplify var_set_display_width().
authorBen Pfaff <blp@cs.stanford.edu>
Mon, 4 Jul 2011 21:49:22 +0000 (14:49 -0700)
committerBen Pfaff <blp@cs.stanford.edu>
Wed, 13 Jul 2011 14:12:57 +0000 (07:12 -0700)
I don't see a reason to keep old_width or to call
dict_var_changed() if nothing changed.

src/data/variable.c

index 41f9221d436c66201393d7bf4efe2b705bce840d..1f9b62e1929114db84b74ad9ee8f517d88d9d085 100644 (file)
@@ -693,14 +693,12 @@ var_get_display_width (const struct variable *v)
 void
 var_set_display_width (struct variable *v, int new_width)
 {
-  int old_width = v->display_width;
-
-  v->display_width = new_width;
-
-  if ( old_width != new_width)
-    dict_var_display_width_changed (v);
-
-  dict_var_changed (v);
+  if (v->display_width != new_width)
+    {
+      v->display_width = new_width;
+      dict_var_display_width_changed (v);
+      dict_var_changed (v);
+    }
 }
 
 /* Returns the default display width for a variable of the given