* varobj.c (install_new_value): Only call value_get_print_value

	if changeable.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 224e89a..38ef25a 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,10 @@
 2007-02-08  Daniel Jacobowitz  <dan@codesourcery.com>
 
+       * varobj.c (install_new_value): Only call value_get_print_value
+       if changeable.
+
+2007-02-08  Daniel Jacobowitz  <dan@codesourcery.com>
+
        Reported by timeless@gmail.com:
        * gdb/target.c (target_flash_erase): Do not return void value.
        (target_flash_done): Likewise.

diff --git a/gdb/varobj.c b/gdb/varobj.c
index 578a4d1..be0f2bc 100644 (file)
--- a/gdb/varobj.c
+++ b/gdb/varobj.c
@@ -953,7 +953,7 @@ install_new_value (struct varobj *var, struct value *value, int initial)
   /* If the type is changeable, compare the old and the new values.
      If this is the initial assignment, we don't have any old value
      to compare with.  */
-  if (initial)
+  if (initial && changeable)
     var->print_value = value_get_print_value (value, var->format);
   else if (changeable)
     {