+
+ Obsolete w65-*-* target.
+ * configure.tgt (w65-*-*): Obsolete.
+ * config/w65/w65.mt: Obsolete.
+ * config/w65/tm-w65.h: Obsolete.
+ * w65-tdep.c: Obsolete.
+ * NEWS: Update. Fix TiC80 description.
+ * TODO: Update.
+
+
+ Obsolete tic80-*-* target.
+ * configure.tgt (tic80-*-*): Obsolete.
+ * config/tic80/tic80.mt: Obsolete.
+ * config/tic80/tm-tic80.h: Obsolete.
+ * tic80-tdep.c: Obsolete.
+ * TODO: Update
+ * NEWS: Update.
+
+
+ * remote-mips.c (S_IROTH): Moved definition from here.
+ * gdb_stat.h (S_IROTH): to here.
+
+ * remote.c (remote_protocol_e, remote_protocol_E): Define.
+ (set_remote_protocol_e_packet_cmd)
+ (set_remote_protocol_E_packet_cmd)
+ (show_remote_protocol_e_packet_command)
+ (show_remote_protocol_E_packet_command): New functions.
+ (init_all_packet_configs): Initialize remote_protocol_e and
+ remote_protocol_E.
+ (remote_resume, remote_async_resume): Support e/E command
+ packets.
+ (show_remote_cmd): Show state of remote_protocol_e and
+ remote_protocol_E.
+ (_initialize_remote): Add "set remote step-over-range-packet"
+ and "set remote step-over-range-w-signal-packet" to CLI.
+
+
+ Obsolete ns32k-*-mach3*, ns32k-umax-*, ns32k-utek-sysv* and
+ ns32k-utek-*.
+ * Makefile.in (ALLDEPFILES): Delete umax-xdep.c and ns32km3-nat.c.
+ (umax-xdep.o, ns32km3-nat.o): Obsolete.
+ * configure.tgt (ns32k-*-mach3*, ns32k-utek-sysv*, ns32k-utek-*):
+ Obsolete.
+ * configure.host (ns32k-*-mach3*, ns32k-umax-*, ns32k-utek-sysv*):
+ Obsolete.
+ * config/ns32k/merlin.mh: Obsolete.
+ * config/ns32k/merlin.mt: Obsolete.
+ * config/ns32k/xm-merlin.h: Obsolete.
+ * config/ns32k/tm-merlin.h: Obsolete.
+ * config/ns32k/nm-umax.h: Obsolete.
+ * config/ns32k/umax.mh: Obsolete.
+ * config/ns32k/umax.mt: Obsolete.
+ * config/ns32k/xm-umax.h: Obsolete.
+ * umax-xdep.c: Obsolete.
+ * config/ns32k/ns32km3.mh: Obsolete.
+ * config/ns32k/ns32km3.mt: Obsolete.
+ * config/ns32k/tm-ns32km3.h: Obsolete.
+ * config/ns32k/xm-ns32km3.h: Obsolete.
+ * ns32km3-nat.c: Obsolete.
+ * ns32k-tdep.c (merlin_skip_prologue): Obsolete.
+ (merlin_frame_num_args): Ditto.
+ * NEWS: Update.
+ * TODO: Update.
+
+
+ * config/djgpp/fnchange.lst: Tweak due to
+ expect/example/chesslib.c.
+
+
+ * m88k-tdep.c (examine_prologue): Change type of ``insn'' from
+ unsigned int to unsigned long. Also, fix format string.
+ (pop_frame): Fix PC_IN_CALL_DUMMY() call so that it doesn't
+ use an undeclared variable. Also, delete declaration and
+ initialization of ``fp'' because it is otherwise unused.
+ * remote-bug.c (sleep): Delete declaration.
+ (bug_store_register, bug_write_memory, bug_read_memory)
+ (bug_insert_breakpoint, bug_remove_breakpoint): Fix sprintf()
+ format statements and cast certain sprintf() arguments to
+ eliminate warnings.
+ (bug_load): Likewise for call to printf_filtered().
+ * config/m88k/tm-m88k.h (FIX_CALL_DUMMY): Eliminate
+ assignment to ``pc''. The necessary assignment is done in
+ hand_function_call() after the invocation of FIX_CALL_DUMMY().
+
+
+ * symtab.c (completion_list_add_name): Remove duplicate string checks,
+ readline already does this, and it's much faster at it, too.
+
+
+ * go32-nat.c: Minor tweaks, to pacify the ari script.
+
+
+ * i386-tdep.c (i386_extract_struct_value_address): New function.
+ * config/i386/tm-i386.h (EXTRACT_STRUCT_VALUE_ADDRESS): Redefine
+ in terms of i386_extract_struct_value_address.
+ (i386_extract_struct_value_address): New prototype.
+
+ * i386-linux-nat.c (i386_linux_dr_get): Change type of return
+ value to `unsigned long'. Change type of `value' to `unsigned
+ long'.
+ (i386_linux_dr_set): Change type of second argument to `unsigned
+ long'.
+ (i386_linux_dr_set_control): Change type of first argument to
+ `unsigned long'.
+ (i386_linux_dr_get_status): Change type of return value to
+ unsigned long.
+ * config/i386/nm-linux.h (i386_linux_dr_set_control,
+ i386_linux_dr_get_status): Adjust prototypes accordingly.
+
+
+ * ia64-linux-nat.c (ia64_linux_stopped_by_watchpoint): Change
+ TRAP_HWBKPT constant to match that in the kernel headers for
+ Linux/IA-64.
+
+
+ * i386bsd-nat.c: Include "gdb_assert.h".
+ [HAVE_PT_GETDBREGS] (DBREG_DRX): Define if not
+ already defined.
+ [HAVE_PT_GETDBREGS] (i386bsd_dr_set, i386bsd_dr_set_control,
+ i386bsd_dr_set_addr, i386bsd_dr_reset_addr,
+ i386bsd_dr_get_status): New functions.
+ * config/i386/nm-fbsd.h [HAVE_PT_GETDBREGS]
+ (I386_USE_GENERIC_WATCHPOINTS): Define.
+ Include "i386/nm-i386.h".
+ (I386_DR_LOW_SET_CONTROL, I386_DR_LOW_SET_ADDR,
+ I386_DR_LOW_RESET_ADDR, I386_DR_LOW_GET_STATUS): New macros.
+ (i386bsd_dr_set_control, i386bsd_dr_set_addr,
+ i386bsd_dr_reset_addr, i386bsd_dr_get_status): New prototypes.
+ * acconfig.h (HAVE_PT_GETDBREGS): New configure macro.
+ * configure.in: Cleanup a few comments. Check for PT_GETDBREGS
+ ptrace request.
+ * config.in, configure: Regenerate.
+
+ * i386-nat.c (i386_insert_aligned_watchpoint): Set address
+ register before enabling it by setting the control register.
+ (i386_remove_aligned_watchpoint): Reset address register after
+ disabling it by setting the control register.
+
+ * i386-tdep.c (i386_extract_return_value): If the type of the
+ return value is TYPE_STRUCT and the number of fields is one, call
+ ourselves with TYPE set tp the type of the first field.
+ (i386_store_return_value): Likewise.
+ This fixes a problem with returning structs consisting of a single
+ `float' or `double' on *BSD.
+
+
+ * lin-lwp.c (lin_lwp_resume): Don't mark LWP as not stopped until
+ we're absolutely sure we're going to resume it.
+
+ * thread-db.c (check_event): Don't report an error if we encounter
+ a thread creation event for a thread that's already in the thread
+ list, since that may legitemately happen. Instead only call
+ attach_thread if it's not already in the thread list.
+
+
+ * config/pa/xm-hppah.h (malloc): Really delete declaration
+ (MALLOC_INCOMPATIBLE): Really delete macro.
+ * cli/cli-cmds.c (apropos_command): Use xcalloc.
+
+
+ * configure.in (build_warnings): Add -Wuninitialized.
+ * configure: Regenerate.
+
+ * v850-tdep.c (v850_scan_prologue): Initialize ``insn2''.
+ * sparc-tdep.c (sparc64_push_arguments): Add default case to
+ switch.
+ * sh-tdep.c (sh_do_fp_register): Replace ``?:'' printf format
+ expression with if statement.
+ * mn10200-tdep.c (mn10200_analyze_prologue): Initialize
+ ``stack_size''.
+ * mips-tdep.c (show_mipsfpu_command): Add default case to switch.
+ (mips_dump_tdep): Fix setting of ef_mips_arch.
+ * mcore-tdep.c (mcore_analyze_prologue): Initialize ``fp_regnum''.
+ * dsrec.c (make_srec): Always initialize ``binbuf''.
+ * monitor.c (monitor_read_memory_single): Do not pass an
+ uninitialized buffer to monitor_error. Make ``i'' more local.
+
+
+ * MAINTAINERS: Change ns32k target to ,-Werror.
+
+ * ns32k-tdep.c: Include "gdbcore.h"
+ (flip_bytes): Change first argument to void*. Add forward
+ declaration.
+ (sign_extend): Add declaration.
+ (merlin_frame_num_args): Add final else clause to if-else chain.
+ (umax_frame_num_args): Ditto.
+ * config/ns32k/tm-umax.h (ns32k_localcount): Declare.
+ (flip_bytes): Ditto.
+
+
+ * i386-linux-tdep.c (FRAMELESS_SIGNAL): Moved here from
+ config/i386/nm-linux.h (including comment).
+ (i386_linux_frame_chain): New function implementing guts of the
+ former FRAME_CHAIN macro, but using read_memory_unsigned_integer
+ instead of read_memory_integer.
+ (i386_linux_frame_saved_pc): Removed doc pointing to
+ i386/tm-linux.h for an explanation of FRAMELESS_SIGNAL since that
+ now lives in this file.
+ * config/i386/tm-linux.h (FRAMELESS_SIGNAL): Moved to
+ i386-linux-tdep.c (including comment).
+ (FRAME_CHAIN): Redefined in terms of i386_linux_frame_chain.
+ (i386_linux_frame_chain): New prototype.
+
+ * gdbserver/low-linux.c [I386_GNULINUX_TARGET]: Cleanup code and
+ make it work again.
+ (NUM_FREGS): Redefine to 0.
+ (NUM_REGS): Redefine as NUM_GREGS.
+ (i386_register_byte, i386_register_raw_size): Update from
+ i386-tdep.c. Add comment about their origin.
+ (regmap, register_u_addr): Take these from i386-linux-nat.c.
+ (i386_register_u_addr): Removed.
+
+ * i386-linux-nat.c (i386_linux_dr_get): Return 0 if ptrace call
+ fails instead of calling perror_with_name. This should fix
+ debugging remote i386 targets with a native Linux/x86 GDB. Add
+ FIXME for this hack.
+
+
+ * arch-utils.c (#include "gdbthread.h"): Removed.
+ (#include "symfile.h"): Removed.
+ (XMALLOC): Removed unused macro.
+ * breakpoint.c (tbreak_command): Removed local declaration.
+ (awatch_command, do_enable_breakpoint, set_breakpoint_count):
+ Remove duplicate declarations.
+ (bpstat_should_step, bpstat_have_active_hw_watchpoints)
+ (remove_solib_event_breakpoints): Fix indentation botch.
+ * c-typeprint.c (#include "command.h"): Removed.
+ (#include "gdbcmd.h"): Removed.
+ * ch-exp.c (ch_terminal_match_float_literal, parse_expr)
+ (parse_primval, parse_untyped_expr, parse_opt_untyped_expr):
+ Removed duplicate declarations.
+ * ch-typeprint.c (#include "command.h"): Removed.
+ (#include "gdbcmd.h"): Removed.
+ * corefile.c (#include "frame.h"): Removed
+ (#include "symfile.h"): Removed.
+ (#include "language.h"): Removed.
+ * dbxread.c (#include "command.h"): Removed.
+ * environ.c (#include "gdbcore.h"): Removed.
+ * event-loop.c (#include "top.h"): Removed.
+ * f-typeprint.c (#include "command.h"): Removed.
+ (#include "gdbcmd.h"): Removed.
+ (#include "language.h"): Removed.
+ (#include "typeprint.h"): Removed.
+ (#include "frame.h"): Removed.
+ * gdbtypes.h (print_type_scalar): Removed declaration.
+ * infcmd.c (#include "completer.h"): Removed.
+ * language.c (#include "frame.h"): Removed.
+ * m2-typeprint.c (#include "command.h"): Removed.
+ (#include "gdbcmd.h"): Removed.
+ (#include "language.h"): Removed.
+ * m2-valprint.c (#include "valprint.h"): Removed.
+ * p-typeprint.c (#include "command.h"): Removed.
+ (#include "gdbcmd.h"): Removed.
+ * p-valprint.c (#include "typeprint.h"): Removed.
+ * parse.c (#include "linespec.h"): Removed.
+ * regcache.c (#include "frame.h"): Removed.
+ * remote.c (#include "frame.h"): Removed.
+ (getpkt_sane): Make static.
+ * source.c (#include "completer.h"): Removed.
+ * stack.c (#include "symfile.h"): Removed.
+ (#include "objfiles.h"): Removed.
+ * symfile.c (#include "completer.h"): Removed.
+ * tracepoint.c (#include "completer.h"): Removed.
+ * values.c (#include "frame.h"): Removed.
+ * varobj.c (#include "valprint.h"): Removed.
+ * wrapper.c (#include "frame.h"): Removed.
+
+ * memattr.c (create_mem_region): Removed unused variable.
+ * remote-nrom.c: Removed spurious semicolon after init_nrom_ops.
+
+
+ * remote-rdp.c (rdp_set_command_line): Add missing double quote.
+
+
+ * solib-aix5.c (build_so_lib_from_mapfile): Use xfree() instead
+ of free().
+
+
+ * TODO (Cleanups): Remove the item about converting docs to GFDL.
+
+
+ * ia64-tdep.c (slotN_contents, replace_slotN_contents): Change
+ type of ``bundle'' from unsigned char * to char *.
+ (ia64_get_saved_register): Use alloca() to allocate register
+ buffers.
+
+ * solib-aix5.c (build_so_lib_from_mapfile): Fix xasprintf() usage.
+
+
+ * proc-utils.h (procfs_ctl_t): New typedef.
+ * proc-api.c (write_with_trace): Change type of ``opcode'' from
+ long to procfs_ctl_t. Don't assume that the target has defined
+ BREAKPOINT. Handle case in which PCRESET is the same as PCUNSET.
+ * proc-events.c (sys/syscall.h, sys/fault.h): Include conditionally.
+ * procfs.c (sys/fault.h, sys/syscall.h): Include conditionally.
+ (gdb_sigset_t, gdb_sigaction_t, gdb_siginfo_t, gdb_premptysysset)
+ (gdb_praddsysset, gdb_prdelsysset, gdb_pr_issyssetmember):
+ Conditionally define as appropriate for AIX/non-AIX systems. Use
+ these defines/typedefs as appropriate elsewhere in file.
+ (struct procinfo): Change type of saved_sigset and saved_sighold
+ from sigset_t to gdb_sigset_t. Make saved_exitset and
+ saved_entryset pointer variables. Add two new fields, num_syscalls
+ and syscall_names.
+ (DYNAMIC_SYSCALLS): Define when HAVE_PRSYSENT_T is defined.
+ (sysset_t_size, sysset_t_alloc): New functions.
+ (load_syscalls, free_syscalls, find_syscall): New functions for
+ platforms which define DYNAMIC_SYSCALLS.
+ (create_procinfo): Call load_syscalls.
+ (destroy_one_procinfo): Call free_syscalls.
+ (GDBRESET): Don't define twice.
+ (proc_modify_flag): Change type of operation code array `arg'
+ from long to procfs_ctl_t.
+ (proc_stop_process, proc_wait_for_stop, proc_run_process)
+ (proc_set_traced_signals, proc_set_traced_faults)
+ (proc_set_traced_sysentry, proc_set_traced_sysexit)
+ (proc_set_held_signals, proc_clear_current_fault)
+ (proc_set_current_signal, proc_clear_current_signal, proc_set_gregs)
+ (proc_set_fpregs, proc_kill, proc_set_watchpoint): Likewise for `cmd'.
+ (proc_set_traced_sysentry): Dynamically allocate variable sized
+ struct gdb_proc_ctl_pcsentry. Also, free it at function exit.
+ (proc_set_traced_sysexit): Dynamically allocate variable
+ sized struct gdb_proc_ctl_pcsexit. Also, free it at
+ function exit.
+ (proc_get_traced_sysentry, proc_get_traced_sysexit): Add new code
+ for reading the sysset_t struct on AIX5.
+ (procfs_debug_inferior): Don't assume that SYS_exit will be
+ defined. Add new code for finding certain syscalls on AIX5.
+ (syscall_is_lwp_exit, syscall_is_exit, syscall_is_exec)
+ (syscall_is_lwp_create): New functions.
+ (procfs_wait): Restructured code which checks for certain
+ system calls to use the new syscall_is_... functions.
+ (procfs_notice_signals): Account for the fact that saved_entryset
+ and saved_exitset in struct procinfo are now pointers.
+
+
+ * symtab.c (find_pc_sect_line): Revise method used for finding
+ the ending pc.
+
+
+ * i386-linux-tdep.c (i386_linux_sigtramp_saved_pc,
+ i386_linux_sigtramp_saved_sp): Make static.
+ (i386_linux_frame_saved_pc): New function based on the old
+ FRAME_SAVED_PC macro, but use read_memory_unsigned_integer instead
+ of read_memory_integer.
+ * config/i386/tm-linux.h (sigtramp_saved_pc): Remove definition.
+ (i386_linux_sigtramp_saved_pc): Remove prototype.
+ (FRAME_SAVED_PC): Redefine in terms of i386_linux_frame_saved_pc.
+ (i386_linux_frame_saved_pc): New prototype.
+
+
+ * MAINTAINERS: Document m88k as a ``Known problem in 5.1''.
+ * TODO (GDB 5.1 - Cleanups): Update.
+
+
+ * fork-child.c (clone_and_follow_inferior): Delete #ifdef
+ HAVE_VFORK.
+
+
+ * config/i386/tm-symmetry.h (PUSH_ARGUMENTS): #undef.
+
+ * i386-tdep.c (i386_push_arguments, i386_store_struct_return): New
+ functions.
+ * config/i386/tm-i386.h (PUSH_ARGUMENTS): New macro.
+ (STORE_STRUCT_RETURN): Redefine in terms of
+ i386_store_struct_return.
+ (i386_push_arguments, i386_store_struct_return): New prototypes.
+ * config/i386/tm-i386v.h (STORE_STRUCT_RETURN): Remove. It's
+ definition was identical to the definition in "i386/tm-i386.h" so
+ the new definition should suffice too.
+
+
+ * event-loop.c (toplevel) [!HAVE_POLL] [NO_FD_SET]: Remove unused
+ definitions for SELECT, NBBY, FD_SETSIZE, NFDBITS, and MASK_SIZE.
+
+ * config/djgpp/djconfig.sh: Add copyright notice.
+
+ * ser-go32.c (rawclock): Remove prototype; include time.h instead.
+ (ISR, dos_hookirq, isr_t): Convert K&R definition to ANSI C.
+ (top level) <string.h>: Include gdb_string.h instead.
+ (dos_noop, dos_raw, dos_noflush_set_tty_state)
+ (dos_print_tty_state, dos_info): Remove ATTRIBUTE_UNUSED.
+
+ * go32-nat.c (go32_create_inferior): Move the declaration of
+ `environ' to here from the top level.
+ (top level) <_initialize_go32_nat>: Remove redundant prototype.
+ <gdb_string.h>: Include it instead of string.h.
+ (store_register): Call register_buffer instead of accessing
+ registers[] directly.
+ (redir_cmdline_delete, redir_cmdline_parse, redir_to_debugger)
+ (redir_to_debugger, redir_debug_init) [__DJGPP_MINOR__ < 3]: Put
+ the function names at the start of the line.
+ (go32_set_dr): Throw internal_error if the argument is not a valid
+ debug register number.
+ (go32_open, go32_close, go32_attach, go32_detach, go32_resume)
+ (go32_wait, go32_xfer_memory, go32_files_info)
+ (go32_terminal_info): Remove ATTRIBUTE_UNUSED.
+
+
+ * mips-tdep.c (mips_gdbarch_init): Tweak indentation.
+
+
+
+ * coffread.c: Include "gdb_assert.h".
+ (coff_symtab_read): Cast the integer s_sclass to a long before
+ casting it to a pointer.
+
+
+
+ * coffread.c (coff_symtab_read): Initialize
+ ``fcn_first_line_addr''. Check that the ``.bf'' always preceeds
+ the ``.ef'' info.
+ * dbxread.c (find_text_range): Initialize ``start'' and ``end''.
+
+
+ * config/sparc/tm-sp64.h (GDB_MULTI_ARCH): Down grade to
+ GDB_MULTI_ARCH_PARTIAL from two. SOFTWARE_SINGLE_STEP is not
+ multi-arch.
+
+ * gdbarch.sh (SOFTWARE_SINGLE_STEP): Add.
+ * gdbarch.h, gdbarch.c: Re-generate.
+
+ * target.h (SOFTWARE_SINGLE_STEP_P)
+ (SOFTWARE_SINGLE_STEP): Delete macro definitions.
+
+
+ * gdbarch.sh (gdbarch_register_read, gdbarch_register_write): Add.
+ * gdbarch.h, gdbarch.c: Regenerate.
+
+ * regcache.h (regcache_read, regcache_write): Declare.
+ (registers, register_valid, register_buffer): Add note that these
+ interfaces are deprecated.
+
+ * regcache.c: Include "gdb_assert.h".
+ (legacy_write_register_gen): Rename write_register_gen.
+ (legacy_read_register_gen): Rename read_register_gen.
+ (regcache_read, regcache_write): New function.
+ (read_register_gen, write_register_gen): New function.
+ (write_register): Simplify. Use write_register_gen.
+ (read_register): Ditto using read_register_gen.
+ (read_signed_register): Ditto.
+ (read_register_bytes): Ditto!!!!
+ (supply_register): Add note that CLEANUP_REGISTER_VALUE is being
+ replaced by gdbarch_register_read.
+
+ * TODO (GDB 5.2 - Cleanups): Add list of gdbarch methods to
+ deprecate.
+
+
+ Fix from Dan Berlin:
+
+ * stabsread.c (read_cpp_abbrev): Properly construct the names of
+ virtual function table pointer fields.
+
+
+ * defs.h (xfree, mcalloc, mmalloc, mrealloc, mfree, xmmalloc,
+ xmrealloc): Move existing declarations to the one place and
+ re-order to be consistent.
+ (xmcalloc, xmfree): Declare.
+ (xmmalloc, xmrealoc): Assume ISO-C - use size_t and void* in
+ declaration.
+
+ * utils.c (size_t): Delete #ifdef defining size_t.
+ (mmalloc, mrealloc, mcalloc, mfree): Re-order.
+ (mmalloc, mrealloc, mcalloc): Document as only calls in GDB
+ corresponding malloc, realloc, calloc.
+ (mfree): Call free directly.
+ (xmmalloc, xmrealloc): Clean up. Assume ISO-C.
+ (xmcalloc, xmfree): New functions. Copy old xcalloc and xfree
+ function bodies to here.
+ (xcalloc, xfree): Call xmcalloc and xmfree respectfully.
+
+
+ * fork-child.c (fork_inferior): Make ``argv'', ``exec_file'' and
+ ``shell_file'' static locals.
+
+ * jv-lang.c (java_link_class_type): Initialize ``field'' and
+ ``method''.
+
+ * jv-valprint.c (java_value_print): Initialize ``next_element''.
+
+
+ * config/i386/nm-i386.h: Fix formatting and change reference to
+ i386-tdep.c to i386-nat.c.
+
+
+ * configure.in: Corrected spelling errors.
+ * configure: Regenerated.
+
+
+ * TODO (GDB 5.1 - Cleanups): Add more targets to obsolete.
+
+
+ * Makefile.in (NTSSTART): Delete.
+ (NTSOBS): Delete.
+ (kdb): Delete target.
+ * kdb-start.c, stuff.c: Mark as obsolete.
+ * NEWS: Update.
+
+
+ * config/pa/xm-hppah.h (HPPA_COMPILER_BUG): Delete. GDB only
+ compiles using an ISO-C compiler.
+ (MALLOC_INCOMPATIBLE): Ditto.
+ * linespec.c (decode_line_1): Delete hack to work around
+ HPPA_COMPILER_BUG.
+
* exec.c (xfer_memory): Always initialize section.
+ Fix from Dan Berlin:
+
* linespec.c (find_methods): Whitespace differences aren't
significant in *un*mangled method names. Use strcmp_iw to compare
- them, not STREQ. (Fix from Daniel Berlin.)
+ them, not STREQ.
+ Fix from Dan Berlin:
+
* Makefile.in: Clean up dependencies on ../include/demangle.h.
(demangle_h): New variable.
(jv-typeprint.o, jv-valprint.o, linespec.o): Use it.
+ Fix from Dan Berlin:
+
* 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.