valops.c (value_of_local): New function.

(value_of_this): Use it.
value.h (value_of_local): Declared.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 37c37a9e52d05f992c00c2c792f9d6e262132efc..ace1c49ef8dbe605a62a6deb11dcc27ac552a248 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2002-10-13  Adam Fedor  <fedor@gnu.org>
+
+       * valops.c (value_of_local): New function.
+       (value_of_this): Use it.
+       * value.h (value_of_local): Declared.
+
 2002-10-13  Adam Fedor  <fedor@gnu.org>
 
        * parse.c: (length_of_subexp, prefixify_subexp): Handle

diff --git a/gdb/valops.c b/gdb/valops.c
index eaf429531082beeebfc4aa7a6a60aafdfec1a4da..e91e231d131a8007a9f6c6737ef2ea3fb15ed862 100644 (file)
--- a/gdb/valops.c
+++ b/gdb/valops.c
@@ -3306,18 +3306,17 @@ value_full_object (struct value *argp, struct type *rtype, int xfull, int xtop,
 
 
 
-/* C++: return the value of the class instance variable, if one exists.
+/* Return the value of the local variable, if one exists.
    Flag COMPLAIN signals an error if the request is made in an
    inappropriate context.  */
 
 struct value *
-value_of_this (int complain)
+value_of_local (const char *name, int complain)
 {
   struct symbol *func, *sym;
   struct block *b;
   int i;
-  static const char funny_this[] = "this";
-  struct value *this;
+  struct value * ret;
 
   if (selected_frame == 0)
     {
@@ -3331,7 +3330,7 @@ value_of_this (int complain)
   if (!func)
     {
       if (complain)
-       error ("no `this' in nameless context");
+       error ("no %s in nameless context", name);
       else
        return 0;
     }
@@ -3341,26 +3340,39 @@ value_of_this (int complain)
   if (i <= 0)
     {
       if (complain)
-       error ("no args, no `this'");
+       error ("no args, no %s", name);
       else
        return 0;
     }
 
   /* Calling lookup_block_symbol is necessary to get the LOC_REGISTER
      symbol instead of the LOC_ARG one (if both exist).  */
-  sym = lookup_block_symbol (b, funny_this, NULL, VAR_NAMESPACE);
+  sym = lookup_block_symbol (b, name, NULL, VAR_NAMESPACE);
   if (sym == NULL)
     {
       if (complain)
-       error ("current stack frame not in method");
+       error ("current stack frame does not contain a variable named \"%s\"", name);
       else
        return NULL;
     }
 
-  this = read_var_value (sym, selected_frame);
-  if (this == 0 && complain)
-    error ("`this' argument at unknown address");
-  return this;
+  ret = read_var_value (sym, selected_frame);
+  if (ret == 0 && complain)
+    error ("%s argument unreadable", name);
+  return ret;
+}
+
+/* C++/Objective-C: return the value of the class instance variable,
+   if one exists.  Flag COMPLAIN signals an error if the request is
+   made in an inappropriate context.  */
+
+struct value *
+value_of_this (int complain)
+{
+  if (current_language->la_language == language_objc)
+    return value_of_local ("self", complain);
+  else
+    return value_of_local ("this", complain);
 }
 
 /* Create a slice (sub-string, sub-array) of ARRAY, that is LENGTH elements

diff --git a/gdb/value.h b/gdb/value.h
index efabecb880660c44ddef9b170f0e116aa738ed29..a38b576947bd4b7f24fabda271ab434ed0b3c5f7 100644 (file)
--- a/gdb/value.h
+++ b/gdb/value.h
@@ -566,4 +566,6 @@ extern CORE_ADDR default_push_arguments (int nargs, struct value ** args,
                                         CORE_ADDR sp, int struct_return,
                                         CORE_ADDR struct_addr);
 
+extern struct value *value_of_local (const char *name, int complain);
+
 #endif /* !defined (VALUE_H) */