* generic/gdbtk-bp.c (gdb_set_bp): Generate a breakpoint event
when we create a breakpoint. The hook is no longer used.
(gdb_set_bp_addr): Ditto.

Christopher Faylor <cgf@redhat.com>
* environ.cc (winenv): Always add SYSTEMDRIVE and SYSYEMROOT to win32-style
environment if they don't already exist.

2001-05-11  Ben Elliston  <bje@redhat.com>

* gas-test.scm (cgen-build.sh, gentest): Escape $ with a backslash
when generating allinsn.d from objdump output. Without it, the
testsuite will treat $ as the regular expression for end of line.

2001-05-10  H.J. Lu  <hjl@gnu.org>

* linker.c (_bfd_generic_link_add_one_symbol): Check loop on
indirect symbols.

* generic/gdbtk-bp.c (breakpoint_list, breakpoint_list_size): New
variables. Gdb is forcing us to maintain our own breakpoint
database.
(Gdbtk_Breakpoint_Init): Initialize our breakpoint database.
(gdb_find_bp_at_addr): Use our breakpoint database.
(gdb_find_bp_at_line): Ditto.
(gdb_get_breakpoint_list): Ditto.
(gdb_get_breakpoint_info): Remove deleted breakpoint hack.
(gdbtk_create_breakpoint): Moved here from gdbtk-hooks.c.
Add breakpoint to our breakpoint database.
(gdbtk_modify_breakpoint): Ditto the move.
(gdbtk_delete_breakpoint): Ditto the move.
Remove breakpoint from our database.
(gdb_get_tracepoint_info): Remove delete tracepoint hack.
It's not needed for tracepoints: they were implemented properly.
(gdbtk_create_tracepoint): Moved here from gdbtk-hooks.c.
(gdbtk_modify_tracepoint): Ditto.
(gdbtk_delete_tracepoint): Ditto.
* generic/gdbtk-hooks.c: Include "gdb-events.h".
(gdbtk_create_breakpoint): Moved to gdbtk-bp.c
(gdbtk_modify_breakpoint): Ditto.
(gdbtk_delete_breakpoint): Ditto.
(breakpoint_notify): Ditto.
(gdbtk_create_tracepoint): Ditto.
(gdbtk_modify_tracepoint): Ditto.
(gdbtk_delete_tracepoint): Ditto.
(tracepoint_notify): Ditto.
(report_error): No longer static.
(gdbtk_add_hooks): Create our own event handler
vector and register breakpoint_create, breakpoint_modify,
and breakpoint_delete handlers in gdbtk-bp.c.

Pass the tupple/list parameter.

2001-05-10  Andrew Cagney  <ac131313@redhat.com>

* ui-out.h (make_cleanup_ui_out_begin_end): Declare.
* ui-out.c (struct ui_out_end_cleanup_data): Define.
(do_cleanup_end): New function.  Replace do_list_end.
(make_cleanup_ui_out_end): New function.
(make_cleanup_ui_out_begin_end): New function.
(make_cleanup_ui_out_list_end): Use make_cleanup_ui_out_end.

2001-05-10  Elena Zannoni  <ezannoni@redhat.com>

        * MAINTAINERS: Declare xcoffread.c open to all maintainers,
        and make Kevin Buettner (kevinb@redhat.com) the reference person.

2001-05-10  Elena Zannoni  <ezannoni@redhat.com>

        * proc-api.c (ioctl_with_trace): Fix uninitialized variable.

2001-05-10  Fernando Nasser  <fnasser@redhat.com>

* MAINTAINERS: Add testsuite subdirectory co-maintainers.

Fix -Werror problem with mi-main.c.

Fix ChangeLog cut-n-paste error!

2001-04-12  Elena Zannoni  <ezannoni@redhat.com>

        * config/gdbserver.exp (gdb_load): Handle the case
        in which the arguments to gdbserver are given in the
        baseboard configuration file.
        Also handle the case in which the server needs to do a
        load.

2001-04-12  Elena Zannoni  <ezannoni@redhat.com>

        * config/gdbserver.exp (gdb_load): Handle the case
        in which the arguments to gdbserver are given in the
        baseboard configuration file.
        Also handle the case in which the server needs to do a
        load.

2001-05-10  Michael Snyder  <msnyder@redhat.com>

* remote.c (remote_open_1): Call no_shared_libraries, so that
symbols for shared libraries can be reloaded per session.
(remote_async_open_1): Ditto.

2001-05-10  Michael Snyder  <msnyder@redhat.com>

* remote.c (bin2hex, hex2bin): New functions.  Factor out these
two conversions which are coded for repeatedly in this module.
(remote_threads_extra_info, remote_wait, remote_async_wait,
store_register_using_P, remote_store_registers, remote_write_bytes,
remote_read_bytes, remote_rcmd): Use bin2hex and hex2bin instead
of coding the conversions inline.
(fromhex): Not exported, change from extern to static.

Delete ``info architecture'' command.

General updates for 5.1/5.2 lists.

Jim Ingham stepping back from Arm maintenance.

* generic/gdbtk-cmds.c: Put on diet. All breakpoint-,
tracepoint-, register-, and stack-related functions moved
into separate files.
(call_wrapper): Renamed to "gdbtk_call_wrapper" and export.
Update all callers.
* generic/gdbtk.h: Move all breakpoint-related stuff into
gdbtk-bp.c.
Remove declaration for "call_wrapper". Now in gdbtk-cmds.h
(and called "gdbtk_call_wrapper").
* generic/gdbtk-varobj.c: Include "gdbtk-cmds.h".
* generic/gdbtk-cmds.h: New file.
* generic/gdbtk-bp.c: New file.
* generic/gdbtk-register.c: New file.
* generic/gdbtk-stack.c: New file.

* Makefile.in (SUBDIR_GDBTK_OBS): Add gdbtk-bp.o, gdbtk-register.o
and gdbtk-stack.o.
(SUBDIR_GDBTK_SRCS): Ditto for the sources.
(gdbtk-bp.o): New rule.
(gdbtk-register.o): New rule.
(gdbtk-stack.o): New rule.
(gdbtk-cmds.o): Update dependencies.
(gdbtk.o): Ditto.
(gdbtk-hooks.o): Ditto.
(gdbtk-varobj.o): Ditto.

Don't loose last block during a dma.

2001-05-10  Fernando Nasser  <fnasser@redhat.com>

* varobj.c (c_number_of_children): Fix memory leak. Delete unwanted old
variables, not just unregister them.

2001-05-10  Fernando Nasser  <fnasser@redhat.com>

* varobj.c (c_number_of_children): Check for target type of void*,
not the target type name.  Allow dereferencing char*.

2001-05-10  Fernando Nasser  <fnasser@redhat.com>

* symfile.c (symbol_file_add_main_1): New static function.
Passes the flags arguments to  symbol_file_add() and takes care
of any necessary reinitializations.
        (symbol_file_command): Call symbol_file_add_main_1() instead of
        symbol_file_add().
        (symbol_file_add_main): Ditto.

Fix more fallout from multi-pass relaxation patch.

* gdbint.texinfo (Clean Design and Portable Implementation):
Renamed from "Clean Design".
(Clean Design and Portable Implementation): Document portable
methods of handling file names, and the associated macros.

* gas/cris/rd-dw2-10.d: Tweak for change in gas dwarf2 line number
handling.

Revert a couple of small changes that inadvertently came in via the
2001-05-03 patch.

Minor ptid_t changes for the upcoming phase 3 ptid patch.

    * config/tc-ia64.c (generate_unwind_image): Align the fragment
        beginning a function's unwind info block.

2001-05-09  H.J. Lu  <hjl@gnu.org>

* elflink.h (elf_link_add_object_symbols): Fix a typo. Allocate
versymhdr->sh_size bytes for extversym instead of hdr->sh_size.
Remove the unused veriable `dynver'.

remove erroneous header.

merge from gcc

* path.cc (mount_info::conv_to_win32_path): Treat UNC paths the same as paths
including `:' or `\'.

* fhandler.h (fhandler_termios::echo_erase): Declare new method.
* fhandler_termios.cc (fhandler_termios::echo_erase): New method for echoing
erase characters.
(fhandler_termios::line_edit): Check the echo flag before echoing control
characters (from Kazuhiro Fujieda <fujieda@jaist.ac.jp>).

2001-05-09  Elena Zannoni  <ezannoni@redhat.com>

* sh3-rom.c (_initialize_sh3_rom): Get rid of specific _WINDOWS
  conditional for help with connections through parallel ports,
  given that the actual code for downloading through a parallel port
  is not conditionalized.

* sh-tdep.c: Remove WIN32_WCE conditional. The wince sh target is
  unmaintaned, and probably on its way to obsolescence.

* library/interface.tcl (gdb_quit_hook): Remove. It's unused.
(gdbtk_quit): Do no shutdown window manager and save
preferences here.
(gdbtk_cleanup): This is the proper place to do those
things.

* i386-tdep.c (i386_frame_saved_pc): New function.
* config/i386/tm-i386.h (FRAME_SAVED_PC): Redefine in terms of
i386_frame_saved_pc.
(i386_frame_saved_pc): New prototype.

* include/pthread.h: Remove C++ comment.

* config/tc-mn10300.c (md_apply_fix3): Accept PC-relative relocs.

* i386-tdep.c (i386_register_virtual_type): New function.
(i386_register_convertible): New function.
* config/i386/tm-i386.h (REGISTER_VIRTUAL_TYPE): Redefine in terms
of i386_register_virtual_type.
(REGISTER_CONVERTIBLE): Redefine in terms of
i386_register_convertible.
(i386_register_virtual_type, i386_register_convertible): New
prototypes.

* Makefile.in (mn10300-tdep.o): New rule.

* Makefile.in (gdb_string_h): Define.  Use it throughout.
Some rules were already using this, even though it isn't defined.

* Makefile.in (mn10300-tdep.o): New rule.

* Makefile.in (obstack_h, target_h): Define; these are already
used elsewhere, but have been expanding to the empty string.
(memattr_h): Define; needed by target_h.

* mn10300-tdep.c (mn10300_extract_return_value): Mark this as
static.  (This was accidentally omitted from the earlier patch.)

* mn10300-tdep.c (mn10300_dwarf2_reg_to_regnum): New function.
(mn10300_gdbarch_init): Register it as the architecture's
dwarf2_reg_to_regnum method.

* mn10300-tdep.c (mn10300_dwarf2_reg_to_regnum): New function.
(mn10300_gdbarch_init): Register it as the architecture's
dwarf2_reg_to_regnum method.

Correct and expand handling of `movm' instruction, and register
saves in general.
* config/mn10300/tm-mn10300.h (D0_REGNUM, A0_REGNUM, MDRQ_REGNUM,
MCRH_REGNUM, MCRL_REGNUM, MCVF_REGNUM): New definitions.
(enum movm_register_bits): New enum.
* mn10300-tdep.c (set_movm_offsets): Use symbolic names for the
bits, not hex literals.  Handle the `other', `exreg0', and
`exother' bits.  Correct handling of `exreg1': it saves r4, r5,
r6, and r7, not r2, r3, r4, and r5.
(saved_regs_size): New function.
(mn10300_frame_chain, mn10300_frame_saved_pc): Use it, instead
of computing the same thing inline, incorrectly.

* mn10300-tdep.c (mn10300_gdbarch_init): We do have a
dummy_breakpoint_offset; it's zero.

* mn10300-tdep.c (mn10300_pop_frame): Split the mn10300-specific
stuff out into mn10300_pop_frame_regular, and use
generic_pop_current_frame.  This lets us share code, and also
makes this function's prototype match that expected by gdbarch.
Make this function static.
(mn10300_pop_frame_regular): New function.
(mn10300_gdbarch_init): Register mn10300_pop_frame as the
gdbarch's pop_frame method.
* config/mn10300/tm-mn10300.h (POP_FRAME): Delete definition.
(mn10300_pop_frame): Delete declaration.

Texinfo fix from geoffk.

* mn10300-tdep.c (mn10300_saved_pc_after_call,
mn10300_extract_return_value,
mn10300_extract_struct_value_address, mn10300_store_return_value,
mn10300_use_struct_convention, mn10300_breakpoint_from_pc,
mn10300_frame_chain, mn10300_skip_prologue,
mn10300_push_arguments, mn10300_push_return_address,
mn10300_store_struct_return, mn10300_frame_saved_pc,
mn10300_init_extra_frame_info, mn10300_frame_init_saved_regs):
Make these functions static; they should only be visible to the
outside world as gdbarch methods.

* config/mn10300/tm-mn10300.h (mn10300_find_callers_reg): Delete
unused declaration.

* mn10300-tdep.c (mn10300_gdbarch_init): Put the gdbarch methods
in some rational order.

* mn10300-tdep.c (mn10300_gdbarch_init): Rather than using
generic_pc_in_call_dummy, use pc_in_call_dummy_at_entry_point.

Use gdbarch for most target parameters for the MN10300, rather
than the tm-*.h file.
* config/mn10300/tm-mn10300.h (MAX_REGISTER_VIRTUAL_SIZE,
REGISTER_BYTES, FP_REGNUM, BREAKPOINT_FROM_PC,
FUNCTION_START_OFFSET, DECR_PC_AFTER_BREAK, INNER_THAN,
SAVED_PC_AFTER_CALL, INIT_EXTRA_FRAME_INFO, FRAME_INIT_SAVED_REGS,
FRAME_CHAIN, FRAME_CHAIN_VALID, FRAME_SAVED_PC,
EXTRACT_RETURN_VALUE, EXTRACT_STRUCT_VALUE_ADDRESS,
STORE_RETURN_VALUE, STORE_STRUCT_RETURN, SKIP_PROLOGUE,
FRAME_ARGS_SKIP, FRAME_ARGS_ADDRESS, FRAME_LOCALS_ADDRESS,
FRAME_NUM_ARGS, POP_FRAME, USE_GENERIC_DUMMY_FRAMES, CALL_DUMMY,
CALL_DUMMY_START_OFFSET, CALL_DUMMY_BREAKPOINT_OFFSET,
CALL_DUMMY_LOCATION, FIX_CALL_DUMMY, CALL_DUMMY_ADDRESS,
TARGET_READ_FP, PUSH_RETURN_ADDRESS, PUSH_DUMMY_FRAME,
SAVE_DUMMY_FRAME_TOS, PUSH_ARGUMENTS, PC_IN_CALL_DUMMY,
REG_STRUCT_HAS_ADDR, USE_STRUCT_CONVENTION, GET_SAVED_REGISTER):
Delete definitions.  We register gdbarch methods for these now.
(struct frame_info, struct type, struct value): Delete forward
declarations of these types; they're no longer necessary, since we
don't have function declarations here any more.
* mn10300-tdep.c: #include "arch-utils.h", to get declarations for
some default gdbarch methods.
(mn10300_store_struct_return): Return void, as expected by
gdbarch.
(mn10300_init_extra_frame_info): Take initial `fromleaf' argument,
as expected by gdbarch.
(mn10300_frame_init_saved_regs): Provide dummy definition for
this, as required by gdbarch.
(mn10300_gdbarch_init): Add mn10300_call_dummy_words, as expected
by gdbarch.  Register gdbarch methods or values for all the stuff
removed from tm-10300.h, listed above.

* doc/porting.texi (Doing a GAS port): Replace `cgen_opcode_open'
with `cgen_cpu_open'; documentation had become out of date.
* doc/rtl.texi (Instruction operands): Likewise.

Replace ui_out_list_{begin,end}() with ui_out_{begin,end}().

Use a stack of structures (instead of a simple array) to keep track of
the top list.

2001-05-08  Michael Snyder  <msnyder@redhat.com>

* language.c (longest_local_hex_string_custom): Strlen test is
inverted -- reverse the sense of the test.

* config/tc-m68k.c: Instead of replacing -1 by 64 in assignment to
fx_pcrel_adjust explicitly sign extend when reading it.

* coff-i386.c (coff_i386_reloc): Don't dump core if output_bfd is
NULL or is not COFF.
(coff_i386_rtype_to_howto): Don't dump core if output section
owner is not COFF.

Remove extraneous character in first line.

* cygheap.cc (_cfree): Add regparm attribute.
(_crealloc): Ditto.
* dcrt0.cc (dll_crt0_1): Default to always checking for executable for now.
* dtable.cc (dtable::not_open): Move method.
* dtable.h (dtable): Here.
* exceptions.cc (ctrl_c_handler): Don't expect process group leader to handle a
signal if it doesn't exist.
* fhandler.h (fhandler_base): Make openflags protected.
* localtime.c (tzsetwall): Check for __CYGWIN__ as well as __WIN32__.
* path.cc (path_conv::check): Add some comments.  Change strcat to assignment.
* lib/_cygwin_S_IEXEC.cc (_cygwin_bob__): Eliminate.
* fhandler_tty.cc (fhandler_console::dup): Set controlling terminal if
necessary.
* fhandler_tty.cc (fhandler_tty_slave::dup): Ditto.

merge from gcc

merge from gcc

Fix typo.

More fallout from 2001-03-30 changes.
* config/tc-m68k.c (md_estimate_size_before_relax): Set fr_var
from md_relax_table, and combine some switch cases.

* config/i386/tm-i386v.h (struct frame_info, struct
frame_saved_regs):  Remove declarations.
(i386_frame_num_args): Remove prototype.

Check Mode not Bank in order to determine rocesor mode.

* include/sys/file.h: Revert special X_OK usage.  Just make it a constant.

* libc/include/sys/stat.h: Revert March 3, Cygwin change.
* libc/include/sys/Ditto.h: Ditto.

    * README: New file.

        * configure.in: Check for executable filename extension.
        * Makefile.in: Add filename extensions.
        * configure: Regenerated.

        * defs: Fix typo setting _test(verbose).
        (gdbtk_test_file): New proc to load executables into gdbtk.
        (gdbtk_test_run): New proc to run executables on gdbtk.
        * browser.exp: Don't check if DISPLAY is set: let
        gdbtk_initialize_display do it all for us.
        Call gdbtk_done when finished with tests.
        * c_variable.exp: Ditto.
        * console.exp: Ditto.
        * cpp_variable.exp: Ditto.
        Only run tests if c++ is supported in the configuration.
        * srcwin.exp: Ditto the DISPLAY stuff.
        Accumulate all test results for the end and report them then.

        * browser.test: Use gdbtk_test_file and gdbtk_test_run
        to load and run an executable.
        * c_variable.test: Use gdbtk_test_file and gdbtk_test_run
        to load and run an executable.
        * srcwin.test: Ditto.
        * srcwin1.test: Ditto.
        * srcwin2.test: Ditto.
        * browser.test: (set_search_mode): Use "configure
        -value" not "entryset" to set the combobox's value. We need
        to have the command fired.
        (browser-3.5): Remove trailing whitespace which causes test to
        fail.
        (browser-4.4): Expect something rational. Regexp "\_1$" cannot
        match "func_10"!
        (browser-4.5): Ditto.
        (browser-5.12): Select all files for search to make test
        match description.
        (browser-6.2): Renamed from another test also calledd 6.1.
        (browser-7.1): Delete all breakpoints at once without from_tty.
        Select all entries in the comobox before invoking BP set/delete
        button.
        (browser-7.2): Ditto.

        * list0.c, list1.c, list0.h: New files. Grabbed from gdb.base.
        * srcwin.exp: Use these local files instead of those in gdb.base.

    * lib/gdb.exp (gdbtk_initialize_display): New proc which will
        set up the display for testing.
        (gdbtk_start): Convert all paths to paths that tcl will like.
        Export target information to environment.
        (_gdbtk_xvfb_init): New proc to start Xvfb if available and
        necessary.
        (_gdbtk_xvfb_exit): New proc to kill Xvfb if necessary.
        (to_tcl_path): New proc to convert a given pathname into
        a path acceptible as an argument to a tcl command.
        (_gdbtk_export_target_info): New proc to export target info
        into the environment for gdbtk testing.
        (gdbtk_done): New proc to signal end-of-test.

Drop self as mn10300 maintainer.

Zap some bogus #ifndef _WIN32 - hangover from WinGDB

* cgen asm/disasm

[opcodes/ChangeLog]
2001-05-07  Frank Ch. Eigler  <fche@redhat.com>

        * cgen-dis.in (default_print_insn): Tolerate min<base instructions
        even at end of a section.
        * cgen-ibld.in (extract_normal): Tolerate min!=base!=max instructions
        by ignoring precariously-unpacked insn_value in favor of raw buffer.

[cgen/ChangeLog]
2001-05-07  Frank Ch. Eigler  <fche@redhat.com>

        * iformat.scm (compute-insn-base-mask-length): Rewrite to tolerate
        various-base-length instruction sets.

merge from gcc

merge from gcc

* library/globalpref.itb (build_win): Preserve the
current working directory.

ChangeLog cleanup: move all references to gdbtk stuff to gdb.gdbtk/ChangeLog (which
has been merged with gdb.gdbtk/ChangeLog-gdbtk).

* disassemble.c (disassembler_usage): Remove unused attribute.

* ldgram.y: Fix typo.

* external.h: Fix typo.
* mips.h: Add/Extend many comments with reference to the MIPS ELF64
spec v. 2.4, available at e.g.
ftp://oss.sgi.com/pub/linux/mips/doc/ABI/ELF64.ps.
(EF_MIPS_UCODE): Define.
(EF_MIPS_OPTIONS_FIRST): Define.
(EF_MIPS_ARCH_ASE): Define.
(EF_MIPS_ARCH_ASE_MDMX): Define.
(EF_MIPS_ARCH_ASE_M16): Define.
(SHF_MIPS_ADDR): Renamed SHF_MIPS_ADDR32.
(SHF_MIPS_STRING): Renamed SHF_MIPS_ADDR64.
(SHF_MIPS_NODUPES): Define.
(ELF64_MIPS_R_SSYM): New MIPS ELF 64 relocation info access macro.
(ELF64_MIPS_R_TYPE3): Likewise.
(ELF64_MIPS_R_TYPE2): Likewise.
(ELF64_MIPS_R_TYPE): Likewise.
(OHW_R10KLDL): Define.

* readelf.c (process_unwind): Remove const specifier.

* ecoff.c (bfd_debug_section): Fix initialization.
* elf.c (_bfd_elf_slurp_version_tables): Change maxidx to unsigned, it
is always a positive integer. Cast away sign mismatch.
* elf32-mips.c: Fix misleading comment and typo.
(_bfd_mips_elf_section_from_bfd_section): Remove unused attribute, use
correct data type.
* elflink.c: Fix typo.
(_bfd_elf_create_dynamic_sections): Remove superfluous initialization.
* ecoffswap.h (ecoff_swap_fdr_in): Cast away sign mismatch.

*** empty log message ***

* mn10300.igen: Doc fixes.

* restore.c: Make the code of caller0 correspond to its comment.

Sun May  6 17:05:00 2001  Robert Collins <rbtcollins@hotmail.com>
* thread.h (pthread_cond): New element cond_access to allow atomic broadcasts.
* thread.cc (pthread_cond::pthread_cond): Initialise cond_access.
(pthread_cond::~pthread_cond): Destroy cond_access.
(pthread_cond::Broadcast): Use cond_access.
(pthread_cond::Signal): Use cond_access.
(pthread_cond_wait): Use cond_access.
(pthread_cond_timedwait): Use cond_access.

Consolidate save_inferior_ptid/restore_inferior_ptid implementation to
one source file.

Make ptid_t related changes.

* config/tc-mn10300.c (md_assemble): Subtract operand->shift
from offset in non-pcrel operands too.

Implement attach/detach for multi-threaded programs on Linux.
* thread-db.c (keep_thread_db): Adjust comment.
(deactivate_target): Removed.
(thread_db_new_objfile): Don't call deactivate_target.  Implement
guts of deactivate_target inline instead.
(attach_thread): Call ATTACH_LWP unconditionally if defined.
(thread_db_attach): New function.
(thread_db_detach): Don't call deactivate_target.  Do necessary
cleanup inline instead.  Set inferior_ptid to LWP corresponding to
the current user-level thread.
(thread_db_kill): Set inferior_ptid to LWP corresponding to the
current user-level thread.
(thread_db_create_inferior): Deactivate target vector if
KEEP_THREAD_DB is zero.
(thread_db_mourn_inferior): Don't call deactivate_target.  Do
necessary cleanup inline instead.
(init_thread_db_ops): Initialize to_attach field to
thread_db_attach.
* lin-lwp.c (lin_lwp_mourn_inferior): Remove prototype.
(stop_wait_callback): Add prototype.
(init_lwp_list): Add comment about when to re-initialize the LWP
list.
(lin_lwp_attach_lwp): Only call ptrace for cloned processes.
Avoid adding publicates to the LWP list.  Only mark an LWP as
signalled if it doesn't correspond to a cloned process.
(lin_lwp_attach): Add initial process to the LWP list.  Make sure
it's stopped and fake a SIGSTOP.
(detach_callback): New function.
(lin_lwp_detach): Implement.
(lin_lwp_create_inferior): Don't re-initialize LWP list here.
Call child_ops.to_create_inferior directly instead of via
target_beneath local.
(lin_lwp_mourn_inferior): Call child_ops.to_mourn_inferior
directly instead of via target_beneath local.

* string.h (cygwin_strchr): Make 'static inline' so that things will still work
when optimized.

Do not insert more than MAX_MEM_FOR_RS_ALIGN_CODE bytes.

* symtab.c (lookup_symtab_1, lookup_partial_symtab): Use basename
instead of non-portable search for `/'.  Use FILENAME_CMP instead
of STREQ, to account for case-insensitive filesystems.
(top-level): #include "filenames.h".

* breakpoint.c (check_duplicates): Use the breakpoint's type, not
its address, to decide whether it's a watchpoint or not.  Zero
is a valid code address.
(update_breakpoints_after_exec): Admonishing comments.
* breakpoint.h (struct breakpoint): Doc fixes.

* breakpoint.c (check_duplicates): Take a breakpoint object as an
argument, rather than an address and section.  All callers changed.