(varobj_update): Remove unused local. Use gdb_assert

to check changelist is non-NULL.  Call error if the frontend tries
to update a non-root variable.

diff --git a/gdb/varobj.c b/gdb/varobj.c
index a2fd1c8bbd356d8ef1ed189b040f404bd2617365..f86082eca5ee9a92b3cbcbb2065563c7b6d6c207 100644 (file)
--- a/gdb/varobj.c
+++ b/gdb/varobj.c
@@ -1037,7 +1037,6 @@ int
 varobj_update (struct varobj **varp, struct varobj ***changelist)
 {
   int changed = 0;
-  int error = 0;
   int type_changed;
   int i;
   int vleft;
@@ -1051,13 +1050,10 @@ varobj_update (struct varobj **varp, struct varobj ***changelist)
   struct frame_info *fi;
 
   /* sanity check: have we been passed a pointer?  */
-  if (changelist == NULL)
-    return WRONG_PARAM;
+  gdb_assert (changelist);
 
-  /*  Only root variables can be updated...  */
   if (!is_root_p (*varp))
-    /* Not a root var.  */
-    return WRONG_PARAM;
+    error (_("Only root variables can be updated"));
 
   if (!(*varp)->root->is_valid)
     return INVALID;