Update stored rendition of varobj value when format changes.

	* varobj.c (varobj_set_display_format): Recomputed
        print_value.
        (c_value_of_variable): Return print_value.

2 files changed, 15 insertions, 1 deletions

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index e100a1a..6263471 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,10 @@
+2008-02-04  Vladimir Prus  <vladimir@codesourcery.com>
+
+	Update stored rendition of varobj value when format changes.
+	* varobj.c (varobj_set_display_format): Recomputed
+        print_value.
+        (c_value_of_variable): Return print_value.
+
 2008-02-03  Doug Evans  <dje@google.com>
 
 	* eval.c (evaluate_subexp_standard): Fix type of result of mixed
diff --git a/gdb/varobj.c b/gdb/varobj.c
index 468beea..258e0bf 100644
--- a/gdb/varobj.c
+++ b/gdb/varobj.c
@@ -673,6 +673,13 @@ varobj_set_display_format (struct varobj *var,
       var->format = variable_default_display (var);
     }
 
+  if (varobj_value_is_changeable_p (var) 
+      && var->value && !value_lazy (var->value))
+    {
+      free (var->print_value);
+      var->print_value = value_get_print_value (var->value, var->format);
+    }
+
   return var->format;
 }
 
@@ -2245,7 +2252,7 @@ c_value_of_variable (struct varobj *var)
 
 	    gdb_assert (varobj_value_is_changeable_p (var));
 	    gdb_assert (!value_lazy (var->value));
-	    return value_get_print_value (var->value, var->format);
+	    return strdup (var->print_value);
 	  }
       }
     }