gdb/ChangeLog

   1 2005-01-14  Andrew Cagney  <cagney@gnu.org>
   2
   3         * exceptions.c (exception_fprintf, exception_print): Move printing
   4         of \n from here ...
   5         (print_exception): ... to here.
   6         (print_flush): New function.
   7         (exception_print, exception_fprintf): Replace duplicated flush and
   8         wrap code with call to print_flush.
   9         (print_and_throw): Simplify, by using xstrvprintf, print_flush,
  10         and print_exception.
  11         (do_write): Delete.
  12
  13         * remote-sds.c (interrupt_query): Call throw_reason instead of
  14         throw_exception.
  15         * remote-mips.c (mips_error, mips_kill): Ditto
  16         * ocd.c (interrupt_query): Ditto.
  17         * nto-procfs.c (interrupt_query): Ditto.
  18         * monitor.c (monitor_interrupt_query): Ditto.
  19
  20 2005-01-14  Jeff Johnston  <jjohnstn@redhat.com>
  21
  22         * ia64-linux-nat.c: Include "ia64-tdep.h" to get register definitions.
  23
  24 2005-01-14  Andrew Cagney  <cagney@gnu.org>
  25
  26         * exceptions.h (catch_exceptions): Delete errstring parameter.
  27         (catch_exceptions_with_msg): Ditto.
  28         * exceptions.c (catch_exceptions_with_msg): Update.
  29         (catch_exceptions): Update.
  30         * symfile-mem.c (add_vsyscall_page): Update.
  31         * frame.c (get_current_frame): Update.
  32         * wrapper.c (gdb_value_struct_elt): Update.
  33         * exceptions.c (catch_exceptions): Update.
  34         * thread.c (gdb_list_thread_ids, gdb_thread_select): Update.
  35         * breakpoint.c (gdb_breakpoint_query, gdb_breakpoint): Update.
  36         * remote-fileio.c (remote_fileio_request): Update.
  37
  38         * exceptions.h (exception_fprintf): Declare.
  39         (exception_print): Drop pre_print parameter.
  40         * mi/mi-main.c (mi_execute_command): Update exception_print call.
  41         * cli/cli-interp.c (safe_execute_command): Update exception_print
  42         call.
  43         * remote.c (remote_open_1): Instead of passing an error prefix to
  44         catch_exceptions, use catch_exceptions and exception_fprintf.
  45         (remote_start_remote): Change return type to void.
  46         * breakpoint.c (insert_bp_location): Instead of passing an error
  47         prefix to catch_exceptions, use catch_exceptions and
  48         exception_fprintf.
  49         (insert_catchpoint): Change return type to void.
  50         (break_command_1): Update exception_print call.
  51         * exceptions.c (exception_fprintf): New function.
  52         (print_exception): New function.
  53         (exception_print): Use print_exception.
  54
  55         * utils.c (error_output_message): Delete function.
  56         * defs.h (error_output_message): Delete declaration.
  57
  58         * linespec.c (symtab_from_filename, decode_variable): Use
  59         throw_error instead of error_silent.
  60         * breakpoint.c (do_captured_parse_breakpoint): Change return type
  61         to void.
  62         (break_command_1): Use catch_exception and check the error return
  63         status.
  64         * exceptions.c (throw_error): New function.
  65         (throw_vsilent): Delete function.
  66         * exceptions.h (throw_error): Declare.
  67         (throw_vsilent): Delete declaration.
  68         * utils.c (error_silent): Delete function.
  69         * defs.h (error_silent): Delete declaration.
  70
  71         * mi/mi-main.c (mi_execute_command): Print the exception.
  72         * cli/cli-interp.c (safe_execute_command): Print the exception.
  73         * exceptions.h (exception_print): Declare.
  74         * exceptions.c (struct catcher): Add field print_message.
  75         (catcher_init): Add parameter print_message, store in the catcher
  76         struct.
  77         (print_and_throw): Only print the message when print_message.
  78         (catch_exceptions_with_msg, catch_errors): Pass print_message=1 to
  79         catcher_init.
  80         (catch_exception): Pass print_message=0 to catcher_init.
  81
  82         * varobj.c (varobj_create): Add missing \n.
  83
  84 2005-01-13  Michael Snyder  <msnyder@redhat.com>
  85
  86         * cli/cli-decode.h: Whitespace tweaks.
  87         * wince.c: Whitespace tweaks.
  88         * uw-thread.c: Whitespace tweaks.
  89         * utils.c: Whitespace tweaks.
  90         * tracepoint.[ch]: Whitespace tweaks.
  91         * solist.h: Whitespace tweaks.
  92
  93 2005-01-13  Andrew Cagney  <cagney@gnu.org>
  94
  95         * exceptions.h (struct exception): Make message const.
  96         * mi/mi-main.c (mi_execute_command): Pass result.message directly
  97         to fputstr_unfiltered.
  98
  99 2005-01-13  Michael Snyder  <msnyder@redhat.com>
 100
 101         * stack.c: Whitespace tweaks.
 102         * somsolib.[ch]: Whitespace tweaks.
 103         * solib-aix.c: Whitespace tweaks.
 104         * solib.c: Whitespace tweaks.
 105         * solib-frv.c: Whitespace tweaks.
 106         * solib.h: Whitespace tweaks.
 107         * solib-irix.c: Whitespace tweaks.
 108         * solib-som.c: Whitespace tweaks.
 109         * solib-sunos.c: Whitespace tweaks.
 110         * solib-svr4.[ch]: Whitespace tweaks.
 111         * ser-tcp.c: Whitespace tweaks.
 112         * ser-unix.c: Whitespace tweaks.
 113         * serial.h: Whitespace tweaks.
 114         * scm-valprint.c: Whitespace tweaks.
 115         * rs6000-tdep.c: Whitespace tweaks.
 116         * rs6000-nat.c: Whitespace tweaks.
 117         * remote.c: Whitespace tweaks.
 118
 119 2005-01-13  Andrew Cagney  <cagney@gnu.org>
 120
 121         * defs.h (error_last_message, error_init): Delete declaration.
 122         * utils.c (fatal, vfatal): Call throw_vfatal.
 123         (error, verror): Call throw_verror;
 124         (do_write, error_stream_1): Delete function.
 125         (error_stream): Simplify, call error.
 126         (error_last_message, error_init, gdb_lasterr): Delete.
 127         (error_silent): Simplify, call throw_vsilent.
 128         * mi/mi-interp.c (mi_cmd_interpreter_exec): Dup the message.
 129         * main.c (captured_main): Delete call to error_init.
 130         * exceptions.c (throw_verror, throw_verror)
 131         (throw_vsilent): New functions.
 132         (do_write, print_and_throw): New functions.
 133         (last_message): New global.
 134         (throw_reason): Replace error_last_message with last_message.
 135         (catch_exceptions_with_msg): Dup the message.
 136         * exceptions.h (throw_verror, throw_vfatal, throw_vsilent):
 137         Declare.
 138
 139 2005-01-13  Michael Snyder  <msnyder@redhat.com>
 140
 141         * remote-st.c: Whitespace tweaks.
 142         * remote-sim.c: Whitespace tweaks.
 143         * remote-mips.c: Whitespace tweaks.
 144         * pa64solib.[ch]: Whitespace tweaks.
 145         * mdebugread.c: Whitespace tweaks.
 146         * main.c: Whitespace tweaks.
 147         * interps.c: Whitespace tweaks.
 148         * infrun.c: Whitespace tweaks.
 149         * infcmd.c: Whitespace tweaks.
 150         * hpux-thread.c: Whitespace tweaks.
 151         * hppa-hpux-tdep.c: Whitespace tweaks.
 152         * gdbcore.h: Whitespace tweaks.
 153         * gdbcmd.h: Whitespace tweaks.
 154         * gdb-events[.c, .h, .sh]: Whitespace tweaks.
 155         * corefile.c: Whitespace tweaks.
 156         * completer.c: Whitespace tweaks.
 157         * coff-solib.c: Whitespace tweaks.
 158         * coff-solib.h: Whitespace tweaks.
 159         * cli-out.c: Whitespace tweaks.
 160         * breakpoint.c: Whitespace tweaks.
 161         * cli/cli-interp.c: Tweak comments, per coding standard.
 162
 163 2005-01-13  Andrew Cagney  <cagney@gnu.org>
 164
 165         * breakpoint.c (gdb_breakpoint_query): Update, use
 166         catch_exceptions_with_msg.
 167         (do_captured_breakpoint): Add uiout parameter.
 168         (gdb_breakpoint): Update, use catch_exceptions_with_msg.
 169         * mi/mi-main.c (mi_cmd_thread_select): Pass mi_error_message to
 170         gdb_thread_select.  Return MI_CMD_ERROR instead of
 171         MI_CMD_CAUGHT_ERROR.
 172         (mi_cmd_thread_list_ids): Ditto for gdb_list_thread_ids.
 173         (enum captured_mi_execute_command_actions): Delete
 174         EXECUTE_COMMAND_DISPLAY_ERROR.
 175         (captured_mi_execute_command): Delete code handling
 176         MI_CMD_CAUGHT_ERROR.
 177         (mi_execute_command): Don't check for
 178         EXECUTE_COMMAND_DISPLAY_ERROR.
 179         * mi/mi-cmd-break.c (breakpoint_notify): Update call to
 180         gdb_breakpoint_query.
 181         (mi_cmd_break_insert): Pass mi_error_message to gdb_breakpoint.
 182         * mi/mi-cmds.h (enum mi_cmd_result): Delete MI_CMD_CAUGHT_ERROR.
 183         * gdb.h (gdb_thread_select, gdb_list_thread_ids)
 184         (gdb_breakpoint, gdb_breakpoint_query): Add error_message
 185         parameter.
 186         * thread.c (thread_command): Update call.
 187         (gdb_thread_select, gdb_list_thread_ids): Update, call
 188         catch_exceptions_with_msg.
 189
 190         * mi/mi-main.c (mi_execute_command): Use catch_exception,
 191         eliminate call to error_last_message.
 192         (captured_mi_execute_command): Change return type to void.
 193
 194 2005-01-12  Andrew Cagney  <cagney@gnu.org>
 195
 196         * exceptions.c (exception_none): New variable.
 197         * Makefile.in: Update dependencies.
 198         * interps.c: Include "exceptions.h".
 199         (interpreter_exec_cmd, interp_exec): Update to return "struct
 200         exception"
 201         * exceptions.h (no_exception): Declare.
 202         * tui/tui-interp.c (tui_exec): Update to return "struct exception"
 203         * mi/mi-interp.c: Include "exceptions.h".
 204         (mi_cmd_interpreter_exec, mi_interpreter_exec): Update to return
 205         'struct exception".
 206         * cli/cli-interp.c (cli_interpreter_exec, safe_execute_command)
 207         (do_captured_execute_command): Update to use catch_exception.
 208         * interps.h: Include "exceptions.h".
 209         (interp_exec_ftype, interp_exec): Return "struct exception".
 210
 211         * exceptions.h (throw_reason): Rename throw_exception.
 212         (enum errors, struct exception): Define.
 213         (catch_exception_ftype): Define.
 214         (catch_exception, throw_exception): Declare.
 215         * exceptions.c (throw_exception): Rewrite.
 216         (throw_reason): New function.
 217         (struct catcher, catcher_state_machine): Replace "reason" with
 218         "exception", delete "gdberrmsg".
 219         (catch_exception): New function.
 220         (catcher_init): Replace "gdberrmsg" parameter with "exception".
 221         (catch_errors, catch_exceptions_with_msg): Re-implement passing
 222         exception to catcher_init.
 223         * utils.c (error_silent, error_stream_1): Use throw_reason.
 224         (internal_verror, quit): Ditto.
 225         * breakpoint.c (insert_catchpoint, break_command_1): Ditto.
 226         * remote-fileio.c (remote_fileio_ctrl_c_signal_handler): Ditto.
 227         * remote.c (remote_open_1, interrupt_query): Ditto.
 228
 229 2005-01-12  Mark Kettenis  <kettenis@gnu.org>
 230
 231         * i386fbsd-tdep.c: Update copyright year.  Include "gdbcore.h",
 232         "regcache.h", "gdb_assert.h" and "bsd-uthread.h".
 233         (i386fbsd_jmp_buf_reg_offset): New variable.
 234         (i386fbsd_supply_uthread, i386fbsd_collect_uthread): New
 235         functions.
 236         (i386fbsdaout_init_abi): Set supply_uthread and collect_uthread.
 237         * Makefile.in (i386fbsd-tdep.o): Update dependency.
 238         * config/i386/fbsd.mt (TDEPFILES): Add bsd-uthread.o.
 239
 240         * bsd-uthread.h: New file.
 241         * bsd-uthread.c: New file.
 242         * Makefile.in (bsd_uthread_h): New variable.
 243         (ALLDEPFILES): Add bsd-uthread.c.
 244         (bsd-uthread.o): New dependency.
 245
 246         * solib.h Update copyright year.
 247         (struct so_list): Forward declaration.
 248         (solib_read_symbols): New prototype.
 249         * solib.c (solib_read_symbols): New function.
 250         (solib_add): Call solib_read_symbols to read in symbols.
 251         (update_solib_list): Call observer_notify_solib_loaded.
 252
 253         * i386bsd-nat.c: Update copyright year.  Don't include
 254         <sys/param.h> and <sys/user.h>.
 255         (register_u_addr, kernel_u_size): Remove functions.
 256         * config/i386/obsd.mh (NAT_FILE): Remove.
 257         * config/i386/obsdaout.mh (NAT_FILE): Set to solib.h.
 258         * config/i386/nm-obsd.h: Remove file.
 259
 260 2005-01-12  Andrew Cagney  <cagney@gnu.org>
 261
 262         * exceptions.c: Include "gdb_string.h".
 263         (enum catcher_state, struct catcher): Define.
 264         (current_catcher): New global, replaces catch_return;
 265         (catch_return): Delete.
 266         (throw_exception): Use current_catcher->buf;
 267         (catcher_init, catcher_pop, catcher_state_machine): New functions.
 268         (catcher): Delete.
 269         (struct catch_errors_args): Delete.
 270         (do_catch_errors): Delete.
 271         (catch_exceptions): Replace body with call to
 272         catch_exceptions_with_msg.
 273         (catcher): Delete.
 274         (catch_exceptions_with_msg, catch_errors): Re-implement using
 275         catcher_state_machine.
 276
 277         * exceptions.h (enum return_reason, RETURN_MASK)
 278         (RETURN_MASK_QUIT, RETURN_MASK_ERROR, RETURN_MASK_ALL)
 279         (return_mask, throw_exception, catch_exceptions_ftype)
 280         (catch_exceptions_with_msg, catch_errors_ftype, catch_errors)
 281         (catch_command_errors_ftype, catch_command_errors): Move to
 282         exceptions.h.
 283         * exceptions.c, exceptions.h: New files.
 284         * top.c: Do not include <setjmp.h>.
 285         (SIGJMP_BUF, SIGSETJMP, SIGLONGJMP, catch_return)
 286         (throw_exception, catcher, catch_exceptions)
 287         (catch_exceptions_with_msg, struct catch_errors_args)
 288         (do_catch_errors, catch_errors, struct captured_command_args)
 289         (do_captured_command, catch_command_errors): Move to exceptions.c.
 290         * wrapper.c, wince.c, win32-nat.c, utils.c: Include "exceptions.h".
 291         * tui/tui-interp.c, top.c, thread.c, symmisc.c: Ditto.
 292         * symfile-mem.c, stack.c, solib.c, rs6000-nat.c: Ditto.
 293         * remote-sds.c, remote-mips.c, remote-fileio.c: Ditto.
 294         * remote-e7000.c, objc-lang.c, ocd.c: Ditto.
 295         * remote.c, nto-procfs.c, monitor.c, mi/mi-main.c: Ditto.
 296         * main.c, m32r-rom.c, infrun.c, inf-loop.c: Ditto.
 297         * hppa-hpux-tdep.c, frame.c, event-top.c, event-loop.c: Ditto.
 298         * corelow.c, corefile.c, cli/cli-interp.c, breakpoint.c: Ditto.
 299         * ada-valprint.c, ada-lang.c: Ditto.
 300         * Makefile.in (HFILES_NO_SRCDIR, COMMON_OBS): Add exceptions.h and
 301         exceptions.o.  Update all dependencies.
 302
 303 2005-01-11  Mark Kettenis  <kettenis@gnu.org>
 304
 305         * config/i386/nm-fbsd.h: Update copyright year.
 306         Don't include "config/nm-bsd.h".
 307         (KERNEL_U_SIZE): Remove macro.
 308         (kernel_u_size): Remove prototype.
 309
 310         * config/vax/nbsdaout.mh (NAT_FILE): Remove.
 311         * config/vax/nm-nbsdaout.h: Remove file.
 312
 313 2005-01-10  Elena Zannoni  <ezannoni@redhat.com>
 314
 315         * MAINTAINERS: Remove self from sh/sh64 maintainership.
 316
 317 2005-01-10  Andrew Cagney  <cagney@gnu.org>
 318
 319         * MAINTAINERS: List Andrew Cagney and Kevin Buettner as can commit
 320         changes to PowerPC, and Andrew Cagney as lead PowerPC GNU/Linux
 321         maintainer.
 322
 323         * MAINTAINERS: Hans-Peter Nilsson and Orjan Friberg can commit
 324         changes to CRIS.
 325
 326         * MAINTAINERS: Remove "Maintenance Only".  Spell out that CAN
 327         COMMIT means can approve).
 328
 329 2005-01-10  Mark Kettenis  <kettenis@gnu.org>
 330
 331         * vax-nat.c: Update copyright year.  Include "inf-ptrace.h".
 332         (vax_register_u_offset): Make statuc.
 333         (_initialize_vax_nat): Construct and add target vector.
 334         * config/vax/vax.mh (NATDEPFILES): Remove infptrace.o and
 335         inftarg.o.  Add inf-ptrace.o.
 336         * Makefile.in (vax-nat.o): Update dependencies.
 337
 338         * inf-ptrace.h: Update copyright year.  Sync comment with
 339         inf-ptrace.c.
 340         (inf_ptrace_trad_target): New prototype.
 341         * inf-ptrace.c: Update copyright year.  Include "regcache.h" and
 342         "gdb_assert.h"
 343         (inf_ptrace_target): Add comment.
 344         (inf_ptrace+register_u_offset): New variable.
 345         (inf_ptrace_fetch_register, inf_ptrace_fetch_registers)
 346         (inf_ptrace_store_register, inf_ptrace_store_registers)
 347         (inf_ptrace_trad_target): New functions.
 348         * Makefile.in (inf-ptrace.o): Update dependencies.
 349
 350 2005-01-09  Mark Kettenis  <kettenis@gnu.org>
 351
 352         * configure.ac: Provide prerequisite headers when checking
 353         <link.h>, <sys/proc.h> and <sys/user.h>.
 354         * configure: Regenerate.
 355
 356 2005-01-08  Mark Kettenis  <kettenis@gnu.org>
 357
 358         * config/powerpc/obsd.mt (DEPRECATED_TM_FILE): Set to
 359         tm-ppc-eabi.h instead of the removed tm-nbsd.h.
 360
 361 2005-01-07  Mark Kettenis  <kettenis@gnu.org>
 362
 363         * config/ia64/tm-linux.h: Tweak comment.  Update copyright year.
 364         Don't include "tm-ia64.h".
 365         * config/ia64/ia64.mt (DEPRECATED_TM_FILE): Remove.
 366         * config/ia64/tm-ia64.h: Remove.  Move contents ...
 367         * ia64-tdep.h: ... here.  Update copyright year.
 368
 369 2005-01-07  Andrew Cagney  <cagney@gnu.org>
 370
 371         * configure.ac: Rename configure.in, require autoconf 2.59.
 372         * configure: Re-generate.
 373
 374         * configure.tgt: Disable arm*-*-*, thumb*-*-*, strongarm*-*-*,
 375         xscale-*-* and i[34567]86-*-netware* code adding rdi-share or nlm
 376         to configdirs.
 377
 378         * configure.in: Replace configdirs with multiple references to
 379         AC_CONFIG_SUBDIRS.
 380         * configure: Re-generate.
 381
 382 2005-01-05  Andreas Schwab  <schwab@suse.de>
 383
 384         * ia64-tdep.c: Include "osabi.h".
 385         (native_find_global_pointer): Delete.
 386         (ia64_find_global_pointer): Renamed from
 387         generic_elf_find_global_pointer.
 388         (FIND_GLOBAL_POINTER): Delete.  Change all users to call
 389         ia64_find_global_pointer instead.
 390         (ia64_gdbarch_init): Call gdbarch_init_osabi.  Remove references
 391         to ia64_linux_sigcontext_register_address and
 392         ia64_linux_write_pc.  Don't set tdep->find_global_pointer.
 393         (_initialize_ia64_tdep): Call gdbarch_register instead of
 394         deprecated register_gdbarch_init.
 395         (struct gdbarch_tdep): Remove find_global_pointer field and move
 396         to ...
 397         * ia64-tdep.h (struct gdbarch_tdep): ... here.
 398         (ia64_linux_sigcontext_register_address): Remove declaration.
 399         (ia64_linux_getunwind_table): Likewise.
 400         (ia64_linux_write_pc): Likewise.
 401         * ia64-linux-tdep.c: Include "osabi.h".
 402         (ia64_linux_sigcontext_register_address): Make static.
 403         (ia64_linux_write_pc): Likewise.
 404         (ia64_linux_init_abi): New.
 405         (_initialize_ia64_linux_tdep): New.
 406         * Makefile.in (ia64-tdep.o, ia64-linux-tdep.o): Depend on
 407         $(osabi_h).
 408
 409 2005-01-05  Kevin Buettner  <kevinb@redhat.com>
 410
 411         * ia64-tdep.c (ia64_gdbarch_init): Eliminate dependency on
 412         functions in ia64-aix-tdep.c.
 413         * ia64-tdep.h (ia64_aix_sigcontext_register_address): Remove
 414         function defined in ia64-aix-tdep.c.
 415         * ia64-aix-tdep.c: Remove file.
 416         * config/ia64/linux.mt (TDEPFILES): Remove ia64-aix-tdep.o from
 417         this list.
 418         * Makefile.in (ia64-aix-tdep.o): Delete.
 419
 420 2005-01-05  Baurjan Ismagulov  <ibr@ata.cs.hun.edu.tr>
 421
 422         Committed by Andrew Cagney.
 423         * ada-valprint.c, aix-thread.c, alpha-nat.c: I18n markup.
 424         * alphabsd-nat.c, alphanbsd-tdep.c, amd64-linux-nat.c: I18n markup.
 425         * amd64-tdep.c, amd64bsd-nat.c, amd64fbsd-nat.c: I18n markup.
 426         * arch-utils.c, arm-linux-nat.c, arm-tdep.c: I18n markup.
 427         * armnbsd-nat.c, armnbsd-tdep.c, auxv.c, avr-tdep.c: I18n markup.
 428         * aix-thread.c (_initialize_aix_thread): Get rid of the
 429         deprecated_add_show_from_set call.
 430         * alpha-tdep.c (_initialize_alpha_tdep): Ditto.
 431         * arm-tdep.c (_initialize_arm_tdep): Ditto.
 432         * command.h (add_setshow_enum_cmd): Add arguments for returning
 433         new list elements.
 434         * cli/cli-decode.c (add_setshow_enum_cmd): Ditto.
 435         * mips-tdep.c (_initialize_mips_tdep): Modify calls to
 436         add_setshow_enum_cmd.
 437
 438 2005-01-04  Mark Kettenis  <kettenis@gnu.org>
 439
 440         * configure.host (ia64-*-aix*): Remove.
 441         * configure.tgt (ia64-*-aix*): Remove.
 442         * config/ia64/aix.mh: Remove file.
 443         * config/ia64/aix.mt: Remove file.
 444         * config/ia64/tm-aix.h: Remove file.
 445         * ia64-aix-nat.c: Remove file.
 446         * Makefile.in (ia64-aix-nat.o): Remove dependency.
 447
 448         * config/m88k/obsd.mh (NATDEPFILES): Really remove inf-child.o.
 449         Add back inf-ptrace.o.
 450
 451 2005-01-04  Andrew Cagney  <cagney@gnu.org>
 452
 453         * version.in: Change format from 6.3.50_2005-01-04-cvs to
 454         6.3.50.20050104-cvs.
 455
 456 2005-01-04  Andreas Schwab  <schwab@suse.de>
 457
 458         * breakpoint.c (break_command_1): Always free err_msg before
 459         returning when an exception was caught.
 460
 461 2005-01-01  Joel Brobecker  <brobecker@gnat.com>
 462
 463         * hppa-tdep.c (prologue_inst_adjust_sp): Fix small confusion
 464         in register number for addil instruction.
 465
 466 For older changes see ChangeLog-2004.
 467 \f
 468 Local Variables:
 469 mode: change-log
 470 left-margin: 8
 471 fill-column: 74
 472 version-control: never
 473 End: