* ldgram.y (YYDEBUG): Don't define.

Modified Files:
.Sanitize ChangeLog

        * .Sanitize: change name from sparclite-*.c to sparcl-*.c.

* aoutf1.h (aout_32_sunos4_write_object_contents): Handle a
machine type of 68000.
* aoutx.h (NAME(aout,machine_type)): Add new argument unknown.
Set *unknown to true if machine type is really unknown, as opposed
to M_UNKNOWN for the 68000.
(NAME(aout,set_arch_mach)): Change NAME(aout,machine_type) call
accordingly.
* libaout.h (NAME(aout,machine_type)): Add new argument to
prototype.

* config/obj-aout.c (obj_aout_frob_symbol): Warn about an attempt
to put a common symbol in a set.

* som.c (EXEC_AUX_ID): Define based on availablity of HPUX_AUX_ID
or HIUX_AUX_ID.
(som_begin_writing): Use EXEC_AUX_ID instead of HPUX_AUX_ID.
(som_write_armap): Use CPU_PA_RISC1_0 as the magic number.  Note
som.c is careful to always define CPU_PA_RISC1_0.

Conditionalize on mips-idt-ecoff.

Sun Jun 12 16:38:42 1994  Stan Shebs  (shebs@andros.cygnus.com)

* mpw-make.in (C, CFLAGS): Removed definitions.
(ALL_CFLAGS): Define.
Set default rule to use {CC} instead of {C}.

Sun Jun 12 16:30:34 1994  Stan Shebs  (shebs@andros.cygnus.com)

* mpw-mh-mpw (CC): Define ALMOST_STDC.
(CFLAGS): Remove ALMOST_STDC, -mc68881.
(LDFLAGS): add -w.

Sun Jun 12 16:25:36 1994  Stan Shebs  (shebs@andros.cygnus.com)

* mpw-configure (config.status): Write only if changed.
* mpw-config.in (readline): Configure it (not built, just used for
definitions).
* mpw-README: Add/update information.

* config/m68k/dpx2.mh (NATDEPFILES):  Remove duplicate inclusion
of inftarg.o.
* config/m68k/tm-dpx2.h (CANNOT_STORE_REGISTER):  Define to inhibit
writing of floating registers, the dpx2 kernel disallows it.

* config/m68k/dpx2.mh (NATDEPFILES):  Remove duplicate inclusion
of inftarg.o.
* config/m68k/tm-dpx2.h (CANNOT_STORE_REGISTER):  Define to inhibit
writing of floating registers, the dpx2 kernel disallows it.
* irix5-nat.c (LM_ADDR):  The loaded address of the shared library
is contained in o_praw.
* irix5-nat.c (solib_map_sections):  Adjust sections by the
difference between the loaded address and the prelinked address.
* irix5-nat.c (solib_address):  Use LM_ADDR for the loaded start
address.
* mdebugread.c (parse_symbol):  Do not relocate stEnd/scText
symbols, their value is absolute.
* mdebugread.c (parse_partial_symbols):  Handle Irix 5.2 shared
libraries fh->adr fields of zero. Relocate minimal symbol values
upon readin. Relocate non-stabs symbols upon readin.
* mdebugread.c (psymtab_to_symtab_1):  Use pst->textlow for the
start address of the outermost block.
* mdebugread.c (parse_lines, parse_procedure):  Pass in pst
instead of section_offsets and use relocated pst->textlow for
line number and procedure address relocations.

From gmo@MicroUnity.com (Guillermo A. Loyola):
* mdebugread.c (parse_symbol, parse_partial_symbols, cross_ref):
Handle SGI Irix5 stIndirect symbol type.

* symconst.h:  Pick up SGI define for stIndirect.

* history.c:  Swap inclusion of rldefs.h and chardefs.h to avoid
CTRL macro redefinition.

Add weak symbols as an extension to a.out.
* read.c (pseudo_set): Only preserve external bit for OBJ_AOUT and
OBJ_BOUT if not BFD_ASSEMBLER.
* config/aout_gnu.h (N_WEAKU, N_WEAKA, N_WEAKT, N_WEAKD, N_WEAKB):
Define as in ../include/aout/aout64.h.
* config/obj-aout.h (OBJ_SYMFIELD_TYPE): If not BFD_ASSEMBLER,
define as char.
(S_GET_WEAK, S_SET_WEAK): Define if not BFD_ASSEMBLER.
* config/obj-aout.c (obj_pseudo_table): Add "weak".
(obj_emit_symbols): Adjust type of weak symbols.
(obj_aout_weak): New static function.

Add weak symbols as an extension to a.out.
* aoutx.h (sym_in_text_section): Don't define.
(sym_in_data_section, sym_in_bss_section): Likewise.
(sym_is_undefined, sym_is_global_defn): Likewise.
(sym_is_debugger_info, sym_is_fortrancommon): Likewise.
(sym_is_absolute, sym_is_indirect): Likewise.
(translate_from_native_sym_flags): Rewrite for clarity.  Rearrange
arguments and change caller.  Handle weak symbols.
(translate_to_native_sym_flags): Likewise.
(aout_link_check_ar_symbols): Don't ignore weak symbols.  Pull
object in from archive if a weak defintion is found for an
existing undefined symbol.
(aout_link_add_symbols): Put all cases in switch.  Set flags of an
undefined symbol to 0.  Handle weak symbols.
(aout_link_write_symbols): Handle weak symbols.
(aout_link_write_other_symbol): Likewise.
(aout_link_input_section_std): Likewise.
(aout_link_input_section_ext): Likewise.
* sunos.c (sunos_write_dynamic_symbol): Likewise.

Add weak symbols as an extension to a.out.
* aout64.h (N_WEAKU, N_WEAKA, N_WEAKT, N_WEAKD, N_WEAKB): Define.
* stab.def: Update symbol value table.

Modified Files:
breakpoint.c ChangeLog

        * breakpoint.c: fix a syntax error native cc does not like.

Modified Files:
sparclite.mt

* change sparclite-tdep.o to sparcl-tdep.o.

Modified Files:
Makefile.in ChangeLog

        * Makefile.in: change sparclite-tdep.c to sparcl-tdep.c.

Added Files:
  sparcl-tdep.c sparcl-stub.c
Removed Files:
sparclite-tdep.c sparclite-stub.c

* change file name because fisrt 8 chars has to be unique.

Modified Files:
.Sanitize

        * .Sanitize: add sparclite-stub.c sparclite-tdep.c to keep.

Modified Files:
ChangeLog

        * sparclite/Makefile.in: fix INCLUDE_CFLAGS to have {srcdir}/../config.

* config/obj-elf.c (obj_elf_section): Don't set any flags based on
the type of a special section.

Don't sanitize out SPARC v9 stuff.

Restore sparc64 targets clobbered in 1994/04/07 checkin.

* nm.c: HOST_64_BIT was renamed to BFD_HOST_64_BIT.

* emultempl/gld960.em: Pass false for new argument to
ldfile_add_library_path.
* emultempl/gld960c.em, emultempl/lnk960.em: Likewise.

* emultempl/sunos.em: Only look for .so files if doing a dynamic
link.

* objcopy.c (copy_archive): Make the temporary directory in the
same directory as the output BFD, since we may not have write
permission on the current directory.  Set the permissions of the
new directory to 0700, not 0777.

* aoutx.h (NAME(aout,canonicalize_reloc)): Handle .bss section.
(NAME(aout,get_reloc_upper_bound)): Likewise.

* config/ho-sunos.h: Include <stdlib.h>.  Don't declare malloc,
realloc, free, or atol.

* coff-i960.c (coff_i960_reloc_type_lookup): Add BFD_RELOC_CTOR.
* linker.c (_bfd_generic_link_write_global_symbol): Don't assume
the section of a common symbol is not NULL.

* gdb.base/recurse.exp: Misc changes to get tests running again
after Kung's changes to the watchpoint code.
* gdb.base/watchpoint.exp: Likewise.

* hppa-tdep.c (target_read_pc, target_write_pc): Accept (and
ignore) a PID argument.
(hppa_pop_frame): Pass a PID to target_write_pc.
* config/pa/tm-hppa.h (TARGET_READ_PC, TARGET_WRITE_PC): Accept
and pass through a PID argument.
(target_read_pc, target_write_pc): Update prototypes.

    * gas/hppa/reloc/reloc.exp (roundmode test): Tweak expected output
        for SOM to match current testcase.

* pr-5022.ch, pr-5022.exp:  New testcase.
* chillvars.exp:  Remove spurious newline.

Modified Files:
breakpoint.c breakpoint.h ChangeLog
Added Files:
sparclite-tdep.c sparclite-stub.c

        * sparclite-tdep.c: new file, contains hardware breakpoint support
        code.
        * sparclite-stub.c: new file, stub code that add support hardware
        breakpoint support.
        * breakpoint.c: add hardware breakpoint support.
        * breakpoint.h: add new breakpoint type to support hardware
        breakpoint.

Modified Files:
sparclite.mt tm-sparclite.h

        * config/sparc/sparclite.mt: add sparclite-tdep.o.
        * config/sparc/tm-sparclite.h: add hardware breakpoints support
        defiines and code.

Modified Files:
nm-hppab.h

        * config/pa/nm-hppab.h: change interface for target dependent
        code supporting watch point.

Modified Files:
nm-irix4.h

         * config/mips/nm-irix4.h: change interface for target dependent
        code supporting watch point.

Modified Files:
ChangeLog

        * remote-os9k.c (rombuf_command): fix a bug accepting rombug
        output.
        * stabsread.c (read_struct_fields): os9k nested structure does not
        have terminating ';', instead it just get to ',' and bit position
        and length.

* Makefile.in:  Bunch of fixes so it actually works in this
directory.  (E.g. add extra ../ where needed.)
Also, add .exe to executables, so we can use suffix rules.
* chexp.exp (test_print_reject):  Update syntax error message.
* chillvars.ch (module PR_5020):  Moved from here ...
* pr-5022.ch: ... to this new file.
* chillvars.exp, pr-5020.exp (binfile):  Add .exe extension.
* chillvars.exp, pr-5020.exp: Don't check all_flag.
* pr-5020.exp:  Add more tests; fix "print y pretty" output.

cvs server works fine on the RS/6000 these days. Only turned on for real
rs6000, though, rather than power and lynx.

Modified Files:
remote-os9k.c stabsread.c

        * remote-os9k.c (rombuf_command): fix a bug accepting rombug
        output.
        * stabsread.c (read_struct_fields): os9k nested structure does not
        have terminating ';', instead it just get to ',' and bit position
        and length.

* scripttempl/i960.sc: Add CONSTRUCTORS to .data.

(nlm_symfile_read):  Use SYMBOL_CLASS to access symbol class.

* Makefile.in (check): Delete ld.new dependency so that a regression
  test doesn't trigger a rebuild of the linker.

* nlmread.c (nlm_symtab_read):  Clean up a bit.
* (nlm_symfile_read):  Record bounds of main() so that backtrace
command will know where to stop.
* objfiles.c (objfile_relocate):  Relocate entry point/func info
for backtrace as well.
* objfiles.h:  Define values for invalid PCs for entry point info.
* symfile.c (init_entry_point_info):  Initialize invalid values
with aforementioned macros.
* config/alpha/tm-alphanw.h:  Turn on FRAME_CHAIN_VALID_ALTERNATE
to cause backtrace to stop when it gets back to main().
* config/i386/tm-i386nw.h:  Ditto.

* configure: Remove temporary files on receipt of a signal.

* ldlang.c (map_input_to_output_sections): For lang_address, call
init_os if it hasn't already been called.

* aoutx.h (aout_get_external_symbols): Don't try to read the
strings if there are no symbols.
(aout_link_write_other_symbol): Use the output section when
working out the type.

Wed Jun  8 12:12:17 1994  Stan Shebs  (shebs@andros.cygnus.com)

* config/udi-gdb.exp, config/unix-gdb.exp: Call default_gdb_start.
* lib/gdb.exp (default_gdb_start): Define.

* Makefile.in (check): Delete as.new dependency, so that
regression test doesn't trigger an assembler build.

rs6k lynx changes

Tue Jun  7 18:45:39 1994  Stan Shebs  (shebs@andros.cygnus.com)

* mpw-config.in (target_arch): Compute from canonical target.
(m68k, mips, powerpc, sparc): Add architectures.
* mpw-make.in (disassemble.c.o): Add.
(ALL_CFLAGS): Remove special flags (-mc68020 -mc68881 -model far).

Move "@" so it's actually on the first line of the shell script in the
makefile target.

* elfcode.h (assign_section_numbers): Put shstrtab, symtab and
strtab sections at end of file.  Avoids bug in some versions of
SVR4 strip.  From Eric Youngdale <eric@tantalus.nrl.navy.mil>.

* configure: If there is a package_makefile_frag, remove
${subdir}/Makefile.tem after copying it in.

* coffcode.h (styp_to_sec_flags): If COFF_PAGE_SIZE is defined,
set SEC_DEBUGGING for STYP_INFO sections.
(coff_compute_section_file_positions): If COFF_PAGE_SIZE is
defined, and D_PAGED is set, set the file position equal to the
section VMA modulo COFF_PAGE_SIZE.
* coffgen.c (coff_real_object_p): If F_EXEC is set, set D_PAGED.
* coff-i386.c: Set D_PAGED in BFD target.
(COFF_PAGE_SIZE): Define.
* coff-m68k.c, coff-sparc.c: Likewise.

* Makefile.in (mostlyclean, realclean): New targets.
* doc/Makefile.in, testsuite/Makefile.in: Likewise.

* gdb.c++/demangle.exp: Use gdb_test_exact, rather than demangle
(which was basically just a local version of the same thing).  In
addition to avoiding duplication, gdb_test_exact is faster.
* lib/gdb.exp (gdb_test_exact): Fix typo.

* lib/gdb.exp (delete_breakpoints): If there were no breakpoints,
don't give an error.
* gdb.base/term.exp: Call delete_breakpoints before starting.

* gdb.base/Makefile.in (EXECUTABLES): Remove t10.
(t10): Add comment.

Reorganized GDB tests - Chill

Reorganized GDB tests - base

Mon Jun  6 18:26:50 1994  Stan Shebs  (shebs@andros.cygnus.com)

Reorganization of the GDB Test Suite.

* gdb.base: New directory, contains all the basic tests.
* gdb.c++: New directory, tests specific to C++.
* gdb.chill: New directory, tests specific to Chill.
* gdb.t00, gdb.t01, gdb.t02, gdb.t03, gdb.t04, gdb.t05, gdb.t06,
gdb.t07, gdb.t08, gdb.t09, gdb.t10, gdb.t11, gdb.t12, gdb.t13,
gdb.t15, gdb.t16, gdb.t17, gdb.t20, gdb.t21, gdb.t22, gdb.t23,
gdb.t24, gdb.t30, gdb.t31: Removed directories, contents moved
into appropriate new directories (t00-t17 to base, t20-t23 to c++,
t30-31 to chill).
* */gdbme.*: Rename to match appropriate expect scripts.
* gdb.base/bitfields.c, gdb.base/break.c, gdb.base/callfuncs.c,
gdb.base/coremaker.c, gdb.base/exprs.c, gdb.base/funcargs.c,
gdb.base/interrupt.c, gdb.base/list0.c, gdb.base/list0.h,
gdb.base/list1.c, gdb.base/mips_pro.c, gdb.base/nodebug.c,
gdb.base/opaque-info.c, gdb.base/opaque0.c, gdb.base/opaque1.c,
gdb.base/printcmds.c, gdb.base/ptype.c, gdb.base/recurse.c,
gdb.base/return.c, gdb.base/run.c, gdb.base/scope0.c,
gdb.base/scope1.c, gdb.base/setvar.c, gdb.base/signals.c,
gdb.base/types.c, gdb.base/twice.c, gdb.base/watchpoint.c,
gdb.base/whatis-info.c, gdb.base/whatis.c: New names of C files.
* gdb.c++/cplusfuncs.cc, gdb.c++/templates.cc, gdb.c++/misc.cc,
gdb.c++/virtfunc.cc: New names of C++ files.
* gdb.chill/chillvars.ch: New name of Chill file.
* gdb.base/configure.in, gdb.c++/configure.in,
gdb.chill/configure.in: Merge in contents of gdb.t*/configure.in.
* gdb.base/Makefile.in, gdb.c++/Makefile.in,
gdb.chill/Makefile.in: Merge in contents of gdb.t*/Makefile.in.
* configure.in (configdirs): Update to reflect directory changes.
(target_abbrev): No longer define for any configuration.
* Makefile.in: Cosmetic improvements to header comments.

* elfcode.h (bfd_section_from_shdr): Don't turn a reloc section
into a BFD section just because SHF_ALLOC is set; require that it
not use the normal symbol table.
(elf_section_from_bfd_section): Corresponding change.

Add lynx targets and hppa flag info

Comment change.

Better indirect and warning symbol handling inspired by Stuart
Quick <stuck@cs.man.ac.uk>.
* linker.c (enum link_action): Add REF, MIND, CWARN, REFC.
(link_action): Change UNDEF_ROW/def and UNDEFW_ROW/def from NOACT
to REF.  Change UNDEF_ROW/indr and UNDEFW_ROW/indr from CYCLE to
REFC.  Change DEF_ROW/indr and COMMON_ROW/indr from CYCLE to MDEF.
Change DEFW_ROW/indr from CYCLE to NOACT.  Change INDR_ROW/indr
from MDEF to MIND.  Change INDR_ROW/warn from WARNC to CYCLE.
Change WARN_ROW/def and WARN_ROW/indr from MWARN to CWARN.  Change
WARN_ROW/com from MWARN to WARN.  Change WARN_ROW/warn from NOACT
to CYCLE.  Change SET_ROW/warn from WARNC to CYCLE>
(_bfd_generic_link_add_one_symbol): Handle REF, MIND, CWARN and
REFC.  If a new indirect symbol has been referenced, push the
reference down to the symbol it points to.  FIx handling of WARN.
* aoutx.h (translate_from_native_sym_flags): If N_WARNING, don't
clobber e_type of next symbol.
(translate_to_native_sym_flags): Likewise.
(aout_link_write_symbols): Loop on bfd_link_hash_warning as well
as bfd_link_hash_indirect.

* bfdlink.h (struct bfd_link_hash_entry): Remove written field:
not needed for all backends.

* libaout.h (struct aout_link_hash_entry): New field written.
* aoutx.h (NAME(aout,link_hash_newfunc)): Initialize written.
(aout_link_write_symbols): Use written, not root.written.
(aout_link_write_other_symbol): Likewise.
* sunos.c (sunos_scan_dynamic_symbol): Likewise.
* libecoff.h (struct ecoff_link_hash_entry): New field written.
* ecoff.c (ecoff_link_hash_newfunc): Initialize written.
(ecoff_link_write_external): use written, not root.written.
* genlink.h (struct generic_link_hash_entry): New field written.
* linker.c (_bfd_link_hash_newfunc): Don't initialize written.
(generic_link_hash_newfunc): Initialize written.
(_bfd_generic_link_output_symbols): Use written, not root.written.
(_bfd_generic_link_write_global_symbol): Likewise.
(_bfd_generic_reloc_link_order): Likewise.

* libecoff.h (ecoff_data_type): Add linker field.
* ecoff.c (ecoff_write_object_contents): Check new tdata linker
field, rather than outsymbols being non-NULL, to decide whether to
output the symbols and relocs.
(ecoff_bfd_final_link): Set new tdata linker field to true.

* config/tc-mips.c (RDATA_SECTION_NAME): Define.
(macro): Correct M_LI_SS SVR4_PIC/EMBEDDED_PIC case.  After M_LI_D
or M_L_DOB or label dob, force a new frag to avoid getting
confused in tc_gen_reloc.
(mips_ip): Use RDATA_SECTION_NAME, not .rdata.
(s_change_sec): Likewise.

* ecoff.c (ecoff_bfd_copy_private_bfd_data): Don't try to copy
data to a non-ECOFF file.

* libbfd-in.h: Add warning that libbfd.h is a generated file.
* libbfd.h: Rebuilt.

Changes to support ELF strip and objcopy on dynamically linked
files.
* elfcode.h (elf_fake_sections): Add prototype.
(bfd_section_from_shdr): Make a BFD section from an SHT_HASH
section, and from an SHT_DYNSYM section, and from the dynamic
string table section.
(elf_object_p): Set D_PAGED if there is a program header.
(elf_make_sections): Remove.
(fix_up_strtabs): Remove.
(elf_fake_sections): Rewrite.  Now sets sh_entsize.
(assign_section_numbers): Rewrite.  Now sets sh_link and sh_info
for all sections.
(elf_compute_section_file_positions):  Don't call obsolete
functions elf_make_sections or fix_up_strtabs.
(swap_out_syms): Set sh_addralign to FILE_ALIGN rather than 4.
(NAME(bfd_elf,write_object_contents)): Permit writing DYNAMIC
objects.
(elf_section_from_bfd_section): Treat SHT_DYNSYM like other normal
sections.  If an SHT_REL or SHT_RELA section is allocated or uses
an unusual symbol table, permit a BFD section to map to it.
Permit most SHT_STRTAB sections to have a BFD section mapped to
them.
(elf_bfd_final_link): Don't set sh_link, sh_info or sh_entsize
fields of dynamic sections here; do it in assign_section_numbers.
* elf32-target.h, elf64-target.h: Add D_PAGED to permitted object
flags.

* elf.c (_bfd_elf_make_section_from_shdr): Only set SEC_DATA if
SEC_LOAD is set, rather than checking SEC_ALLOC.

* libbfd-in.h (bfd_realloc): Change last arg to "size_t size".
(Move libbfd.h change over to source file).

Remove a couple of myteriously duplicated entries.

keep configure.bat

keep files for pushw test

Fix value_print, which used to be ostensibly langauge-indepentdent,
but would print pointers and arrays in C syntax.  Instead, call
a language-specific function.  See ChangeLog for details.

* configure.in:  Use mh-ncrsvr43.  Patch from
Tom McConnell <tmcconne@sedona.intel.com>.

* mh-ncrsvr43:  New file from Tom McConnell <tmcconne@sedona.intel.com>.

* read.c (emit_expr): Use memset to zero out memory, rather than
going through md_number_to_chars.  This permits handling symbolic
arguments when the size is larger than sizeof (valueT), if
TC_CONS_FIX_NEW is prepared to handle the case (as it is on MIPS).

Various changes to gdb.t31.

* config.guess (i386-unknown-bsdi): New system to guess.
* config.guess (i386-unknown-bsdi):  No longer need to
check #if defined(__bsdi__) && defined(__i386__).

binutils.texi: (objdump) fix formatting in options summary

note renaming

actually do c++filt.1 -> cxxfilt.man renaming on trunk...

* as.c (show_usage), config/tc-alpha.c (md_show_usage),
config/tc-mips.c (md_show_usage): Fix up messages.

improve usage message

* as.h: Replace flagseen with separate variables.
* as.c (parse_args): Set them.  Don't accept -1 option, or -v
explicitly (it's a synonym for --version).
* as.c, input-scrub.c, messages.c, read.c, symbols.c, write.c,
config/obj-aout.c, config/obj-aout.h, config/obj-bout.c,
config/obj-bout.h, config/obj-coff.c, config/obj-coff.h,
config/obj-vms.c, config/tc-hppa.c, config/tc-i386.c,
config/tc-i960.c, config/tc-m68k.c, config/tc-mips.c,
config/tc-vax.c: Use the new flag variables instead of flagseen.
* config/tc-vax.c [OBJ_VMS]: Recognize -+, -1, -v, and document in
usage.

don't sanitize out v9 stuff any more

* som.c (som_write_object_contents): Don't compute the file header's
checksum here.
(som_write_headers): Instead do it here.

* as.c (show_usage): Remove target specific messages;
instead, call md_show_usage.
(parse_args): Use getopt_long_only.  Take pointers to argc and
argv.
(main): Pass parse_args pointers.
* as.h: Remove 3 variables that are redundant with flagseen.
* as.c, messages.c: Change their users to use flagseen.
Define getopt stuff.
* tc.h: Update md_parse_option decl.  Add md_show_usage decl.
* config/tc-*.c: Add md_shortopts, md_longopts,
md_longopts_size, md_show_usage.  Change calling convention for
md_parse_option.  Remove md_parse_long_option.
* config/tc-ns32k.c: Rename `struct option' to `struct ns32k_option'.
* config/tc-i386.h: Don't define md_parse_option.

* main.c: Move entire file except for #ifndef MAIN_OVERRIDE code
to new file top.c.  Make things extern instead of static and
similar rearrangements to deal with this.
* top.h: New file.
* utils.c: Move fputs_unfiltered to main.c.  Remove
FPUTS_UNFILTERED_OVERRIDE ifndef.
* Makefile.in: Change so that gdb uses main.c, utils.c, and top.c,
and libgdb uses utils.c and top.c.

    * libbfd.h (bfd_realloc): Change last arg to "size_t size".

* objdump.c (display_target_list): Remove unused local ok.