* linespec.c (find_methods): Just call CHECK_TYPEDEF on t, rather

than asking for sym_class's type; that's circuitous.  Remove
sym_class, since the last use of it is gone.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index ccc9cef165e453a0b2041b351ecb5626a420bd42..90085db490d63648616f9d550df39b936fd96050 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2001-03-20  Jim Blandy  <jimb@redhat.com>
+
+       * linespec.c (find_methods): Just call CHECK_TYPEDEF on t, rather
+       than asking for sym_class's type; that's circuitous.  Remove
+       sym_class, since the last use of it is gone.
+
 2001-03-20  Eli Zaretskii  <eliz@is.elta.co.il>
 
        * breakpoint.c (print_it_typical) <bp_access_watchpoint> [UI_OUT]:

diff --git a/gdb/linespec.c b/gdb/linespec.c
index d70d63426db86d71a3dd70c5ded5f69c63a0f566..6359bae4935a359fbee5fc611d19c3c638314876 100644 (file)
--- a/gdb/linespec.c
+++ b/gdb/linespec.c
@@ -104,23 +104,19 @@ find_methods (struct type *t, char *name, struct symbol **sym_arr)
 {
   int i1 = 0;
   int ibase;
-  struct symbol *sym_class;
   char *class_name = type_name_no_tag (t);
 
   /* Ignore this class if it doesn't have a name.  This is ugly, but
      unless we figure out how to get the physname without the name of
      the class, then the loop can't do any good.  */
   if (class_name
-      && (sym_class = lookup_symbol (class_name,
-                                    (struct block *) NULL,
-                                    STRUCT_NAMESPACE,
-                                    (int *) NULL,
-                                    (struct symtab **) NULL)))
+      && (lookup_symbol (class_name, (struct block *) NULL,
+                        STRUCT_NAMESPACE, (int *) NULL,
+                        (struct symtab **) NULL)))
     {
       int method_counter;
 
-      /* FIXME: Shouldn't this just be CHECK_TYPEDEF (t)?  */
-      t = SYMBOL_TYPE (sym_class);
+      CHECK_TYPEDEF (t);
 
       /* Loop over each method name.  At this level, all overloads of a name
          are counted as a single name.  There is an inner loop which loops over
@@ -168,7 +164,7 @@ find_methods (struct type *t, char *name, struct symbol **sym_arr)
                  }
                else
                  phys_name = TYPE_FN_FIELD_PHYSNAME (f, field_counter);
-
+               
                /* Destructor is handled by caller, dont add it to the list */
                if (DESTRUCTOR_PREFIX_P (phys_name))
                  continue;