gcc lint

gcc lint

some gcc lint

* config/obj-elf.c (obj_elf_section): Set SEC_LOAD when using
default flag values.

Back out this change; it doesn't work.

Tue Sep  7 13:06:44 1993  Jim Kingdon  (kingdon@lioth.cygnus.com)

* dbxread.c (process_one_symbol): If the value of an N_FUN for a
function is zero, use minimal symbols to get the address.

* stabs.texinfo (Stabs In ELF, Statics): More on relocating stabs
in ELF files.

Revert previous change.  It doesn't work because it neglects to
consider there may be a file compiled without -g screwing things up
(e.g. we are looking for the minimal symbol for "main" and we get the
one for "start" instead).

This is the change I mean:
* minsyms.c, symtab.h (lookup_next_minimal_symbol): New function.
* dbxread.c (process_one_symbol): Use it.

* minsyms.c, symtab.h (lookup_next_minimal_symbol): New function.
* dbxread.c (process_one_symbol): Use it.

* Makefile.in (mipsread.o): Depend on $(bfd_h).

add comment

* stabs.texinfo (Stabs In ELF, Statics): More on relocating stabs
in ELF files.

revise comment

    * Makefile.in:  add TEXINPUTS variable and use it in ld.dvi target

support for --host HOST

Add missing ifdef to make m68k-aout targets happy.

* configure.in (m88k-*-sysvr4*): Comment out previous change.

* remote.c (remote_wait): Don't call error.  Instead, call warning
inside a loop.  User can ^C to get out.

* gdbserver/Makefile.in (TAGS): config files are in
$(srcdir)/../config, not $(srcdir)/config.

* config/pa/tm-hppa.h: Declare target_read_pc and target_write_pc.
(STORE_RETURN_VAULE): Pass the correct offset of the return
register to write_register_bytes.
* hppa-tdep.c: Use target_write_pc if PCOQ_TAIL_REGNUM was not saved.

* gdb.t15/{return.c,return.exp,Makefile.in}:
New test, currently commented out.

* config.sub: Remove h8300hhms alias.

* stabs.texinfo (Stabs In ELF): Talk about N_FUN value.

* config/m68k/tm-m68k.h (FIX_CALL_DUMMY): Changed name of swapping
routine to match BFD name change.
* config/z8k/tm-z8k.h (FIX_CALL_DUMMY): Likewise.

* libbfd-in.h: Removed swapping routines declarations.
* bfd-in.h: Added swapping routine declarations, and renamed them
from _do_* to bfd_*.
* libbfd.c: Renamed swapping routines from _do_* to _bfd_*.
* Changed all callers.
* libbfd.h, bfd-in2.h: Rebuilt.

* mips-dis.c, m88k-dis.c: Don't include libbfd.h.  Changed calls
to BFD swapping routines to correspond to BFD name changes.

* ecoff.h (AUX_GET_ANY, AUX_PUT_ANY): Changed to reflect further
change in bfd swapping routine names.

* dbxread.c (process_one_symbol): If the value of an N_FUN for a
function is zero, use minimal symbols to get the address.

Changes to support stabs-in-coff

* elf32-i386.c (elf_howto_table): Set pcrel_offset to true for
R_386_PC32.

* ecoff.h (AUX_GET_ANY): Change name of _do_getb32 to reflect bfd
changes.

* stabs.texinfo (Local Variable Parameters): Talk about nameless
parameters on VAX.

Changed elf32_symbol_type to elf_symbol_type

Snapshot of stand alone i386 NLM debugging stub.  Currently sanitized
out because it contains Novell proprietary code.

* remote.c (remote_wait): Added 'W' and 'N' responses.

* nlmconv.c (total_bss_size): Removed.
(main): Set the bss vma to always follow the data section.  Move
symbols into new sections, and adjust values by output_offset.
(setup_sections): Don't copy all sections, but instead point all
text sections to .text, all data sections to .data, and all bss
sections to .bss.
(copy_sections): Adjust accordingly.

@up/@down -> @raisesections/@lowersections

* ld.texinfo: re-enable included config file; conditionalize doc
for -oformat to interact properly with SingleFormat doc config
var; rename @up/@down to @raisesections/@lowersections.

* main.c, utils.c: Add comments about immediate_quit.

tiny additions & fixes

PA and ELF changes

obj-elf.h change

more Utah changes, some cleanup of mine

(elf_symbol): Fixed name of elf_symbol_type.
No longer conditionalized on sparcv9.

* elfread.c (elf_symtab_read): Don't add symbols starting with ".L"
to minimal symbols.

configure.in, libbfd, elf, elf-hppa (utah) changes

Reorganized setting of makefile variables so 64-bit stuff works again.

more changes from Utah, see log for details

Deleted some fields of elf*_symbol_type and elf_obj_tdata that were getting
set but not used.  Merged elf*_symbol_type into a single elf_symbol_type.

regenerated after doc fix

(_do_get*, _do_put*): Renamed via macros to clean up namespace.

* stabs.texinfo: Make info author notice match the TeX author notice.

* elfread.c (elf_symtab_read): Don't add symbols starting with ".L"
to minimal symbols.

* stabs.texinfo: Many minor cleanups.

* stabs.texinfo: Many minor cleanups.

remove obsolete comment about call function stuff being duplicated

* target.c (pop_target): Don't try to deal with the stack becoming
empty.  Shouldn't happen and the code that tried was broken.

* dcache.c: Cast return value from xmalloc.

* remote.c: Move setting of immediate_quit from remote_open to
remote_start_dummy and set it back to zero when done.

* m88k-tdep.c: Remove a bunch of unused #includes.

* libhppa.h (hppa_field_adjust): Remove unused `init_value' variable.

* language.h: Add comment about current_language.

Reverted previous change, and added this one instead:
* config/tc-mips.c (macro_build): Accept 'z', and ignore it.
(macro): Use "z,s,t" for div instructions to match corresponding
change in opcode table.
(mips_ip): Added 'z'--must be zero register.

* mips-opc.c: Change div machine instruction to be z,s,t rather
than s,t.  Change div macro to be d,v,t rather than d,s,t.
Likewise for divu, ddiv, ddivu.  Added z,s,t case for drem, dremu,
rem and remu which generates only the corresponding div
instruction.  This is for compatibility with the MIPS assembler,
which only generates the simple machine instruction when an
explicit destination of $0 is used.
* mips-dis.c (print_insn_arg): Handle 'z' (always register zero).

* mips-tdep.c (_initialize_mips_tdep): Change heuristic-fence-post
from var_uinteger to var_zinteger.

Only build nlmconv if configured for a NetWare target.
* configure.in: If we have some *-*-netware* target, or are using
all targets, set BUILD_NLMCONV to $(NLMCONV_PROG) in Makefile.
* Makefile.in (PROGS): Use $(BUILD_NLMCONV) rather than
$(NLMCONV_PROG).

* config/tc-mips.c (macro): If $0 is destination of divide, just
generate the simple machine instruction, for compatibility with
the MIPS assembler.

* mips-opc.c: Move div machine instruction after macro forms.
Change d,s,t form to d,v,t.  Likewise for divu, ddiv and ddivu.
This is for compatibility with the MIPS assembler, which only
generates the simple machine instruction when an explicit
destination of $0 is used.

* m88k-tdep.c: Include sys/dir.h before sys/user.h.
Include sys/types.h regardless of USG.

* configure.in: Fix typo (delta88r4 -> delta88v4).

* config/m88k/xm-delta88.h: Don't include sys/siginfo.h.  It was
to make this work on SVR4 before SVR4 had its own configuration,
and it breaks SVR3.

* config/m88k/delta88v4.h (NATDEPFILES): Remove infptrace.o inftarg.o.

* hosts/delta88v4.h: New for SVR4.
* configure.hosts: Use it.

* config/m88k/xm-dgux.h: Renamed from config/m88k/xm-m88k.h.
* config/m88k/m88k.mh: Use xm-dgux.h.
* config/m88k/xm-m88k.h: New file, with HOST_BYTE_ORDER,
MAKEVA_END and MAKEVA_ARG.
* config/m88k/xm-*.h: Include m88k/xm-m88k.h.
* printcmd.c: Remove __INT_VARARGS_H code; now in xm-m88k.h.

new for /proc support on svr4 delta88

* printcmd.c: Remove __INT_VARARGS_H code; now in xm-m88k.h.

* printcmd.c: Remove __INT_VARARGS_H code; now in xm-m88k.h.

* config/m88k/xm-dgux.h: Renamed from config/m88k/xm-m88k.h.
* config/m88k/xm-m88k.h: New file, with HOST_BYTE_ORDER,
MAKEVA_END and MAKEVA_ARG.
* config/m88k/xm-*.h: Include m88k/xm-m88k.h.
* printcmd.c: Remove __INT_VARARGS_H code; now in xm-m88k.h.

renamed from xm-m88k.h

* remote-udi.c (udi_wait): Call `warning' not `error'.

* remote-udi.c (udi_wait): Call `warning' not `error'.

* symtab.c (COMPLETION_LIST_ADD_SYMBOL): If the symbol has a
demangling, don't put the mangled form in the completion list.

* symtab.c, symfile.c, c-exp.y, ch-exp.y, m2-exp.y, buildsym.c,
symfile.h, stabsread.c, minsyms.c, solib.c, nlmread.c, dwarfread.c
partial-stab.h, symmisc.c, gdbtypes.c: Lint.  Remove (or put
inside #if) unused variables and labels.  Fix unclosed comment.
Deal with enumeration values unhandled in switch statements.  Make
sure non-void functions return values.  Include appropriate
headers.
* dbxread.c (elfstab_build_psymtabs): Don't check for unsigned
value < 0.

Load the udi library procedures.

* symtab.c (find_pc_line, find_line_common),
symtab.h (struct linetable), xcoffread.c (arrange_linetable):
Revise comments re linetable sorting.
* buildsym.c (compare_line_numbers): Sort by pc, not by line.
* coffread.c: Tell end_symtab to sort the line table.

* coffread.c: Re-work a lot of the coff-specific stuff to use stuff
in buildsym.c.  This includes coff_finish_block, coff_context_stack,
coff_local_symbols, coff_file_symbols, coff_global_symbols,
coff_end_symtab and coff_add_symbol_to_list.
(read_enum_type): Deal with it now that we have a "struct pending"
not a "struct coff_pending".

* buildsym.c (end_symtab): Don't realloc subfile->linetable.

* coffread.c: Re-work a lot of the coff-specific stuff to use stuff
in buildsym.c.  This includes coff_finish_block, coff_context_stack,
coff_local_symbols, coff_file_symbols, coff_global_symbols,
coff_end_symtab and coff_add_symbol_to_list.
(read_enum_type): Deal with it now that we have a "struct pending"
not a "struct coff_pending".

* coffread.c: Re-work a lot of the coff-specific stuff to use stuff
in buildsym.c.  This includes coff_finish_block, coff_context_stack,
coff_local_symbols, coff_file_symbols, coff_global_symbols,
coff_end_symtab and coff_add_symbol_to_list.
(read_enum_type): Deal with it now that we have a "struct pending"
not a "struct coff_pending".

index -> strchr

* remote.c: Add comment explaining why dcache is disabled.
(remote_fetch_word, remote_store_word): Make static and #if 0.
They are not called from anywhere.

Wed Sep  1 11:35:49 1993  Jim Kingdon  (kingdon@lioth.cygnus.com)

* symtab.c (find_pc_line, find_line_common),
symtab.h (struct linetable), xcoffread.c (arrange_linetable):
Revise comments re linetable sorting.
* buildsym.c (compare_line_numbers): Sort by pc, not by line.
* coffread.c: Tell end_symtab to sort the line table.

* remote.c: Add comment explaining why dcache is disabled.
(remote_fetch_word, remote_store_word): Make static and #if 0.
They are not called from anywhere.

* symtab.c (find_pc_line, find_line_common),
symtab.h (struct linetable), xcoffread.c (arrange_linetable):
Revise comments re linetable sorting.
* buildsym.c (compare_line_numbers): Sort by pc, not by line.
* coffread.c: Tell end_symtab to sort the line table.

bzero -> memset

bcmp -> memcmp

* symtab.h (struct linetable), xcoffread.c (arrange_linetable):
Revise comments re linetable sorting.
* buildsym.c (compare_line_numbers): Sort by pc, not by line.
* coffread.c: Tell end_symtab to sort the line table.

* coffread.c: Re-work a lot of the coff-specific stuff to use stuff
in buildsym.c.  This includes coff_finish_block, coff_context_stack,
coff_local_symbols, coff_file_symbols, coff_global_symbols,
coff_end_symtab and coff_add_symbol_to_list.
(read_enum_type): Deal with it now that we have a "struct pending"
not a "struct coff_pending".

* buildsym.c (end_symtab): Don't realloc subfile->linetable.

bcopy -> memcpy

write.c, configure.in changes

(write_relocs) [RELOC_EXPANSION_POSSIBLE]: Declare tc_gen_reloc correctly.

Deleted explicit "return" statements without values at the ends of functions.

* libaout.h, netbsd386.c: Change M_I386 to M_386_NETBSD.  M_I386
is an SCO pre-define.

add an alias for data section type

break dcache out of remote-nindy.c

* mipsread.c (parse_partial_symbols):  Use language from FDR if it
is unambigous.  Patch from ptf@delcam.co.uk (Paul Flinders).
* mipsread.c (ecoff_symfile_info):  New struct to hold the global
pending_list.
* mipsread.c (mipscoff_symfile_init, parse_partial_symbols):
Allocate the global pending list and link it to the objfile.
* mipsread.c (is_pending_symbol, add_pending):  Use global pending
list from objfile. Allocate pending list entries from the
psymbol_obstack.
* mipsread.c (free_pending):  Remove. The pending list is now
freed when the psymbol_obstack is freed.
* mipsread.c (psymtab_to_symtab1):  Remove pending list allocation,
the global pending list is used now.
* mipsread.c (parse_partial_symbols):  Skip only the first
file indirect entry when building the dependency list.

break dcache code out of remote.c

pulling dcache code out of remote.c