Lint Updates

Lint stuff which didn't make it last time

Fri Jun 19 20:23:21 1992  Steve Chamberlain  (sac@thepub.cygnus.com)
Lints for bfd_reloc_code_type->bfd_reloc_code_real_type, and
correct calling of howto special functions and fixes from Raeburn
for gas<>bfdness
* archures.c: nuke bfd_reloc_code_type
* bout.c, cpu-h8300.c, coff-i960.c, coff-m88k.c: special function lint
* coffcode.h (styp_to_sec_flags): STYP_INFO is marked as
SEC_NEVER_LOAD, various other bfdgas newness
* reloc.c->libbfd.h: change protype of bfd_default_reloc_type_lookup
* targets.c: change jump table vector to above

(find_source_lines):  Handle large st_size fields.

COFF changes for dealing better with EPI 29K C compiler output.

* coffread.c (record_minimal_symbol):  Pass the minsym type to it.
Callers changed.
(coff_end_symtab):  Sort blocks if needed.  Complain if misordered.
(read_coff_symtab):  Move patch_opaque_types call from
coff_symfile_read.  Restrict it to symtabs from this objfile.
(process_coff_symbol: C_TPDEF):  Don't put ordinary foward
references on opaque type chain; just let coff_lookup_type handle 'em.
(decode_type):  Complain about tagndx values on
non-struct/union/enum types, which the EPI compiler tends to produce.

* symtab.c (list_symbols):  Make minimal symbol variable-finding work.

* tm-68k.h (FIX_CALL_DUMMY): Avoid alignment and byte order
dependency.

* elfread.c (elf_symfile_read):  Update bfd_elf_find_section
usage to match new prototype.  Include libbfd.h to get prototype.

Typo.

* gdb.texinfo:  document `set remotedebug' and `set rstack_high_address'.

  * infcmd.c (jump_command):  Use cleanups to avoid memory leaks.
  * stack.c (return_command):  Use cleanups to avoid memory leaks.

* remote-adapt.c, remote-eb.c, remote-mm.c:  Lint.  Fix
INT_REGNUM to INTE_REGNUM.
* tm-29k.h (SDB_REG_TO_REGNUM):  Add for EPI 29K C compiler.

* elf.c (bfd_elf_find_section):  Mark as INTERNAL_FUNCTION so
its prototype will be included in libbfd.h.  Change result type
to struct * (rather than equivalent typedef) so it can be used in
the prototype, where they typedef won't be known.
* libbfd.h:  Updated version.

HPPA BS

* sysdep-obsd.h:  #include <sys/types.h> to make this more Kosher.

HPPA merge.

* bfd.h:  HPPA merge.

* configure.in, dbxread.c, hppa-coredep.c, hppa-pinsn.c,
hppabsd-core.c, hppabsd-tdep.c, hppabsd-xdep.c, hppahpux-tdep.c,
hppahpux-xdep.c, munch, partial-stab.h, tm-hppabsd.h,
tm-hppahpux.h, xm-hppabsd.h, xm-hppahpux.h:  HPPA merge.

* Makefile.in (c-exp.tab.c, m2-exp.tab.c):  Filter out bogus extern
declarations of malloc/realloc/free that are inserted by some
versions of yacc.
* m2-exp.y:  Prevent conflicts with TRUE and FALSE tokens by
#undeffing them.
* xm-rs6000.h:  Declare malloc/realloc/free appropriately.  Yet
another decl of strdup (this really ought to come from libiberty.h!).

HPPA merge.

* Makefile.in, archures.c, configure.in, cpu-hppa.c, hppa.c, libhppa.h,
targets.c:  HPPA merge.

* config.sub:  HPPA merge.

* remote.c (getpkt):  Error if input exceeds buffer size.
(_initialize_remote):  `set remotedebug' enables packet trace.

* dbxread.c (process_one_symbol:N_FUN):  GCC now produces relative
N_SLINE's, etc, just like Sun cc on Solaris2.

* am29k-tdep.c (read_register_stack, write_register_stack):
Change RSTACK_HIGH_ADDR to rstack_high_address, a user-settable
variable.  Add `set' and `show' commands for it.
* doc/gdb.texinfo:  Document it.

* eval.c:  Avoid residue-by-zero when evaluating without side effects.
(Bug and fix found by Pierre Willard.)

* sparc-tdep.c:  Clean up slightly for Solaris2.

Fri Jun 19 12:53:28 1992  John Gilmore  (gnu at cygnus.com)

* config/mh-apollo68v, mh-sco, mh-sysv, mh-sysv4}: RANLIB=true.

Mon Jun 15 13:50:34 1992  david d `zoo' zuhn  (zoo at cirdan.cygnus.com)

* configure.in: use mh-sysv4 on solaris2

* configure.in:  Separate Solaris2 from SYSV4 on SPARC.
* config/solaris2.mh:  Kludge around Sun compiler bug.

  * valprint.c (type_print_1):  Plug memory leak.  Print all
  C++ syms as demangled, not just functions.
  * buildsym.c (read_range_type):  When we find a signed char
  type, do a lookup of signed char, not plain char.  Plain char's
  still get looked up as plain char's elsewhere.

1) Better fix for section "Readline Killing Commands": delete entire
first paragraph.  It looks like it's left over from previous
nomenclature; term "cutting" introduced there is not used in rest of
document.  ("killing" is used instead, as in the section title and in
the following paragraph.)

2) More consistent use of @key for things to type in; in a few cases
not having it actually produced wrong doc on paper---notably "M--"
comes out as "M" followed by emdash, if in the plain Roman body
text...

Delete dangling "There is a" at end of first paragraph under "Readline
Killing Commands".  None of our saved versions has the rest of this sentence,
whatever it may have been...

* h8300hms.em, h8300hms.sc-sh:  do the right thing for -r
* ldexp.c:  lint
* ldlang.c(open_output): set the target arch and machine as soon
as we can. (lang_size_sections): use new macros for setting vma
* ldwrite.c: lint

* xm-rs6000.h:  Fix decls for malloc, realloc, and free.

* tm-rs6000.h:  Remove #include of inferior.h.  Fixes many
compilation errors.

* xm-rs6000.h:  Add decl for strdup().

* xm-rs6000.h:  Remove #include of inferior.h.  Fixes many
compilation errors.

* breakpoint.c (enable_command, disable_command):  Without args,
should only affect normal breakpoints and watchpoints.

* libaout.h (aout_backend_data):  Change defs of two bitfields to
be unsigned chars instead to get around rs6000 compiler problem.

* elf.c (bfd_section_from_shdr, elf_slurp_symbol_table):
Correct misconception that there can be only one symbol table.
Only call elf_slurp_symbol_table on the full symbol table, not
the dynamic one which is only a subset of the full one.

(The bfd library was slurping in the first ELF symbol table it
found and then ignoring any others.  For dynamically linked
executables, this happened to be the dynamic symbol table, which
is a subset of the full symbol table and only contains the
symbols that need to be used in dynamic linking.)

* m68k-pinsn.c (print_insn_arg):  Make register be const.

* xcoffexec.c:  Remove ' from comment.
* xm-sun3os4.h:  Define MALLOC_INCOMPATIBLE to avoid conflicts
with decls of malloc in c-exp.tab.c (as produced by yacc).
There's got to be a better way to do this...

Don't align sections in I960 coff files.
(I know there's a GDB freeze on, but this only affects writing i960
coff files, and Steve needs it ASAP.)

* partial-stab.h: Convert single rindex use to strrchr.
* mipsread.c, dbxread.c:  Remove troublesome inclusion of non-
standard <strings.h> file, now that the only single use of
rindex in the gdb source files is gone.

* mipsread.c:  Undo ill effects from including <strings.h>,
which #defines index to be strchr.  Unfortunately, index is
a member of a symbol table structure that can't be changed.
* mipsread.c:  tm-mips.h includes coff/symconst.h and coff/sym.h,
remove redundant #include's.

gratuitous white space changes merged from other prep releases

Keep mh-solaris

Add move-if-change to DEVO_SUPPORT_FILES.

Add demangle.h to HFILES.

Unscrew this file...  (Been up too long I think...)

More cleanup in gdb packaging...

Keep sun4sol2.m{th}.

Keep tm-sun4sol2.h and xm-sun4sol2.h.

More 29k crap to remove!

One mo time...

Fixup install.all target so that install-texinfo can be removed properly by GDB
packaging procedures.

Remove 29k/UDI stuff for now.  For the time being, the original Makefile is
called Makefile.in-29k-UDI.

Save Makefile with 29k stuff for later.

Save scripts.

Remove 29k/UDI for now.

* mipsread.c:  #include <strings.h> for rindex().

typo

hitachi bug gix

Bug fixes for Hitachi

  * xcoffexec.c (map_vmap):  Turn comment into a real C comment.

Changes reflecting recent changes in BFD library sources.

* hosts/sparc.h (abort, exit): Hide these names if compiling with
gcc version 2, to avoid warnings.

[time to file a PR on cvs...]

Various changes to get linker working again for a.out:
 * don't set/adjust section file positions or vmas more than once
 * use correct page size and segment size when calculating them
 * deal with some variations in a.out implementations
Tested on sun4 and sun4->sun3 so far, will be testing further but
needed to get wider exposure&testing.  See ChangeLog for details.

Also:
* coffcode.h (coff_write_relocs): Write out swapped reloc, not
pre-swapped version.
* hosts/sparc.h (abort, exit): Hide these names if compiling with
gcc version 2, to avoid warnings.

Various changes to get linker working again for a.out:
 * don't set/adjust section file positions or vmas more than once
 * use correct page size and segment size when calculating them
 * deal with some variations in a.out implementations
Tested on sun4 and sun4->sun3 so far, will be testing further but
needed to get wider exposure&testing.  See ChangeLog for details.

Also:
* coffcode.h (coff_write_relocs): Write out swapped reloc, not
pre-swapped version.
* hosts/sparc.h (abort, exit): Hide these names if compiling with
gcc version 2, to avoid warnings.

* dbxread.c (process_one_symbol), partial-stab.h:  Ignore
extraneous SO stabs from busted C++ compilers.

* config/mh-ncr3000 (INSTALL):  Don't use /usr/ucb/install,
it is broken on ncr 3000's.

* config/mh-ncr3000 (INSTALL):  Don't use /usr/ucb/install,
it is broken on ncr 3000's.
* config/mh-ncr3000 (RANLIB):  Use RANLIB=true.

* config/ncr3000.mh (INSTALL, RANLIB):  Don't use /usr/ucb/install,
it's broken on ncr 3000's.  Use simple "true" for RANLIB.

* Makefile.in (VERSION):  Bump to 4.5.5.
* symtab.c (decode_line_1):  Until C++ support stabilizes, when
C++ lookups fail, print possibly helpful hint about completion.
* cplus-dem.c (demangle_signature):  Fix ARM style demangling
for static data members.
* dbxread.c (dbx_psymtab_to_symtab_1):  Fix prototype.
* config/ncr3000.mh (INSTALL):  Don't use /usr/ucb/install,
it's broken on ncr3000's.

Accept m680?0 for wrs as vxworks68.  Also deleted an unreachable path to wrs.

* dwarfread.c (decode_modified_type):  Change type of first arg.
Change 'modifier' to char from unsigned char.  Cast single use
that needs to be unsigned char.
* symtab.h (SYMBOL_BASEREG_VALID):  Explain disabling.
* utils.c (strdup_demangled):  Add function.
* defs.h (strdup_demangled):  Add prototype.
* stack.c (return_command):  Demangle C++ function names for query.
* infcmd.c (jump_command):  Demangle C++ function names for query.
* cplus-dem.c (consume_count):  New function and prototype.
* cplus-dem.c (demangle_class, gnu_special, demangle_func_type,
do_type):  Replace conversion code with consume_count().
* cplus-dem.c (gnu_special):  Fix demangled of static members.
* source.c (list_command):  Print demangled function names
when appropriate.  Fix supplied by Peter Schauer.

* buildsym.c (define_symbol):  Nameless types are now on several
platforms; generalize them and un-ifdef them to make Solaris 2
work.

* infcmd.c:  Fix typo (reported by Rob Savoye).
* xm-sun4sol2.h (gregset_t):  Thinko in register set definition.
* symtab.h:  Disable SYMBOL_BASEREG_VALID until it works.
* dbxread.c (dbx_psymtab_to_symtab_1):  Renamed from
psymtab_to_symtab_1.  Use current psymtab's sym_offset and symbol_size,
rather than the one for the first file in the dependency chain.

* readline.c: Make new SIGNALS_* macros to parameterize the
ugly changes in signal blocking macro.  Use them throughout,
reducing #ifdef HAVE_POSIX_SIGNALS and HAVE_BSD_SIGNALS clutter
significantly.  Make all such places use POSIX if available,
to avoid losing with poor `sigsetmask' emulation from libiberty.

* readline.c (insert_some_chars):  Return void.

* dbxread.c (end_psymtab):  Only reset texthigh if it's not already
set.  Don't reset our own texthigh, or dependency-only pst's, in
scanning all the rest of the psymtabs.
(process_one_symbol):  Fix comments around N_OBJ, N_OPT, N_UNDF.

* buildsym.h (N_UNDF):  Improve comments.
(N_LSYM, etc):  Skip types without names (":T(0,3)=sfoob...").

* objdump.c (dump_elf_stabs):  Also dump .stab.index and .stab.excl sections.
(dump_elf_stabs_1):  Split out main body of old dump_elf_stabs.
* objdump.1, binutils.texi:  Document new sections dumped.

* symtab.h (struct symbol):  Add aux_value union for preserving
an additional per-symbol value.
* symtab.h (SYMBOL_BASEREG, SYMBOL_BASEREG_VALID):  Add defines.
* frame.h (FRAME_GET_BASEREG_VALUE):  Provide default definition.
* findvar.c (read_var_value):  Use SYMBOL_BASEREG if valid.
* printcmd.c (address_info):  Use SYMBOL_BASEREG if valid.
* symmisc.c (print_symbol):  Use SYMBOL_BASEREG if valid.

Lint

Make `clear_symtab_users_once' non-static so objfiles can be linked.

* elfread.c (record_minimal_symbol_and_info):  Kludged to put
extra info in the minimal symbol.
(elf_symtab_read):  The extra info is the size of an ELF object,
which was kludged to us in the `udata' field of the BFD symbol.
Gag me with a crowbar...
(elf_symfile_read):  Keep track of both the absolute load address,
and the offset between load addr and symbols.  Handle STABS as
well as DWARF sections, passing the absolute load address to
elfstab_build_psymtabs.

* objfiles.c (free_objfile):  Eliminate storage leaks.  Contributed
by <Peter.Schauer@regent.e-technik.tu-muenchen.dbp.de>.
* symfile.c (symfile_bfd_open):  Comment where name is freed.
* symmisc.c (extend_psymbol_list):  Comment where list is freed.

* dbxread.c:  Add symbol_size to struct dbx_symfile_info.
Add symbol_size and file offsets to struct symloc.
Add static symbol_table_offset, string_table_offset,
file_string_table_offset, next_file_string_table_offset,
last_function_name.
(add_old_header_file):  Convert error() to complain()t.
(dbx_symfile_read):  Get symbol size via the objfile.
Also get symbol_table_offset.
(dbx_symfile_init):  Record symbol size and count via the objfile.
(dbx_next_symbol_text):  Always add file_string_table_offset when
accessing string table.
(read_dbx_symtab):  Initialize file_string_table_offset to 0.
(SET_NAMESTRING):  Use the offset.
(end_psymtab):  Large kludge to determine addresses where source
files start and end (for the psymtab header).
(psymtab_to_symtab_1):  Set file_string_table_offset before reading.
Get this, symbol size, and symbol offset, from psymtab.
(read_ofile_symtab):  Don't back up one symbol for Solaris2.
Patch up last_source_start_addr if zero in N_SO.
(process_one_symbol):  Add variable function_start_offset.
( "", N_FUN, N_LBRAC, N_RBRAC, N_SLINE):  Add ifdef for
BLOCK_ADDRESS_FUNCTION_RELATIVE.
( "", N_OBJ, N_OPT):  Ignore.
(elfstab_build_psymtabs):  New function to read stabs out of
an ELF file.

* Makefile.in:  Accept $(BISON) even though we really want $(YACC).
(gdb):  use GLOBAL_CFLAGS when linking, too.
* partial-stab.h (N_UNDF):  Deal with Sol2 relative stringtab offsets.
(N_OBJ, N_OPT):  Ignore.
('f', 'F'):  Save last function name.

* symfile.h:  Prototype elfstab_build_psymtabs.
* symfile.c:  Add almost-OK debug versions of add_psymbol_*to_list.

* xm-sysv4.h:  Add <limits.h> to avoid conflicting defns in defs.h.

Remove reg_names decl (now in defs.h).

Avoid ANSI C "empty translation unit" idiocy.

* section.c (STD_SECTION):  Remove extra semicolon in declarations.
* configure.in:  Rewrite target parsing to use simple format.
Handle sparc-sun-solaris2 configuration.
* aout64.h:  Avoid ANSI C brain death warning.
* elf.c:  Avoid trigraph (???) or /* in comments.

Lint

Lint

Add sun4sol2 => sparc-sun-solaris2.

* buildsym.h:  Add processing_acc_compilation flag.
* buildsym.c (read_sun_builtin_type, read_sun_floating_type):  New.
(define_symbol):  Skip arg types in function entries.  Resolve
overloaded 'P' which acc uses for prototypes of functions called
by this file.
(read_type: 'b', 'r'):  Handle Solaris2 builtin types.

* minsyms.c (prim_record_minimal_symbol_and_info):  Hack to
save size of ELF symbols.  FIXME.
* tm-sun4os5.h:  Rename to tm-sun4sol2.h.  Update defines for Sol2.
* xm-sun4os5.h:  Rename to xm-sun4sol2.h.  Hack more defines.

* config/sun4os5.m[ht]:  Rename to config/sun4sol2.m[ht]; new xm, tm.

* configure.in:  Solaris config is sparc-sun-solaris2.
* config/sun4os5.m[ht]:  Rename to config/sun4sol2.m[ht].

change to new solaris naming system, add sysv4 host config

* expprint.c (print_subexp):  Add missing ']'.
* defs.h (reg_names):  Fix declaration to match that in infcmd.c
* stack.c (reg_names):  Delete redundant (and inconsistent) decl.
* WHATS.NEW:  Point out improved C++ function name handling.
* gdbtypes.c (lookup_fundamental_type):  For now, use the same
type names for both implicitly and explicitly signed integral
types.  See comment in the source code.

Two `long long' fixes from Robert R. Henry (rrh@dino.tera.com):
* defs.h (longest_to_int):  Avoid void arm of ?: in error case.
* expprint.c (print_subexp):  Fix printing of register names.

Latest and greatest, many bug fixes, value annotations, stack frame
annotations, etc...

* objdump.1, binutils.texi:  Update for --stabs.  Also fix
objdump's doc to use -- rather than + for long options.
(FIXME:  Not yet fixed everywhere in binutils.texinfo.)

Small changes from Solaris port.  The big stuff is not all working yet.

* inferior.h (register_valid):  Declare.
* remote-udi.c, rs6000-xdep.c, sparc-xdep.c, tm-rs6000.h:  Remove decl.
* objfiles.h (ALL_OBJFILE_{SYMTABS,PSYMTABS,MSYMBOLS}):  Add
macros for traversing the data structures in a single objfile.
* tm-m88k.h, tm-sparc.h (REGISTER_NAMES):  Remove extra semicolon.
* tm-i960.h (REGISTER_NAMES):  Cosmetic change.
* infcmd.c:  Lint.

#undef PC, which Solaris2 defines in sys/types.h, clobbering the
termcap global variable PC.

New feature: --stabs prints out a .stab section from an ELF file.
Installed under #ifdef ELF_STAB_DISPLAY so it can be easily disabled,
since it requires bfd-internals header files and such.

Add N_OBJ and N_OPT from Solaris-2.

Regen'd from bfd.c after removing elf_core_tdata_struct.

* elf.c (elf_obj_tdata):  Merge elf_obj_tdata_struct and
elf_core_tdata_struct into a single common struct.  Core files
wouldn't have worked at all without this.
(bfd_elf_find_section):  New function for GDB's undercover use
to find string sections that BFD hides from it.
(elf_get_str_section):  Avoid multiple alloc&reads for same data; lint.
(elf_object_p, elf_core_file_p):  Allocate internal file header
storage dynamically.
* bfd.c (union {...} tdata):  Remove elf_core_tdata_struct.
* demo64.c:  Prevent "empty translation unit" warnings from idiots.

* i386.h: a.out magic numbers from
mohring@informatik.tu-muenchen.de