From e26cc3490a00a58329c61e33afd8e682e449b475 Mon Sep 17 00:00:00 2001 From: David Anderson Date: Mon, 23 Oct 2000 22:49:29 +0000 Subject: [PATCH] Corrected spelling errors in comments --- gdb/ChangeLog | 10 ++++++++++ gdb/TODO | 24 ++++++++++++------------ gdb/command.c | 2 +- gdb/config/nm-lynx.h | 2 +- gdb/corelow.c | 2 +- gdb/main.c | 2 +- gdb/mips-tdep.c | 2 +- gdb/remote.c | 2 +- gdb/serial.h | 2 +- gdb/top.c | 8 ++++---- gdb/utils.c | 2 +- 11 files changed, 34 insertions(+), 24 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index a698f1a..ab50cd1 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,11 @@ +2000-10-23 David B Anderson + + * TODO: Correct spelling errors + * command.c (_initialize_command) corelow.c (core_open) + main.c (captured_command_loop) mips-tdep.c (mips32_next_pc) + remote.c serial.h top.c utils.c config/nm-lynx.h: + Correct spelling errors in comments + 2000-10-22 Kevin Buettner * sparc-nat.c (fetch_core_registers): Protoize. @@ -32,6 +40,7 @@ Fri Oct 20 19:08:47 2000 Andrew Cagney * remote-utils.c (gr_multi_scan): Protoize. * remote-vx.c (vx_xfer_memory, net_get_symbols): Protoize. +>>>>>>> 1.717 2000-10-13 Fernando Nasser * remote.c (putpkt_binary): Call read_frame, not getpkt. Log message. @@ -124,6 +133,7 @@ Fri Oct 20 19:08:47 2000 Andrew Cagney * rs6000-tdep.c (skip_prologue): Add new parameter lim_pc. Update all callers. +>>>>>>> 1.709 2000-10-03 Kevin Buettner * remote-bug.c (bug_xfer_memory, bug_insert_breakpoint, diff --git a/gdb/TODO b/gdb/TODO index a6247ca..2cf6bc4 100644 --- a/gdb/TODO +++ b/gdb/TODO @@ -71,7 +71,7 @@ Dan Nicolaescu writes: > stoping in weak functions. > > It stops in a function that is defined as weak, not in the function -> that is actualy run... +> that is actually run... -- @@ -314,7 +314,7 @@ determine the default isa/byte-order. -- -Rely on BFD_BIG_ENDIAN and BFD_LITTLE_ENDIAN instead of host dependant +Rely on BFD_BIG_ENDIAN and BFD_LITTLE_ENDIAN instead of host dependent BIG_ENDIAN and LITTLE_ENDIAN. -- @@ -508,7 +508,7 @@ http://sourceware.cygnus.com/ml/gdb-patches/2000-07/msg00042.html Apart from the d30v, are there any sim/common simulators that make use of this? -A brief summary of what happended is that sim/common/sim-break.c was +A brief summary of what happened is that sim/common/sim-break.c was created as a good idea. It turned out a better idea was to use SIM_SIGBREAK and have GDB pass back sim_resume (..., SIGBREAK). @@ -604,7 +604,7 @@ Document overlay machinery. ``(gdb) catch signal SIGNAL'' -Overlaps with ``handle SIGNAL'' but the implied behavour is different. +Overlaps with ``handle SIGNAL'' but the implied behavior is different. You can attach commands to a catch but not a handle. A handle has a limited number of hardwired actions. @@ -806,7 +806,7 @@ Rename read_register{,_pid}() to read_unsigned_register{,_pid}(). If / when GDB starts to support the debugging of multi-processor (rather than multi-thread) applications the symtab code will need to -be updated a little so that several independant symbol tables are +be updated a little so that several independent symbol tables are active at a given time. The other interesting change is a clarification of the exact meaning @@ -902,7 +902,7 @@ did have a frame pointer register. Modern architectures typically construct a virtual frame-handle from the stack pointer and various other bits of string. -Unfortunatly GDB still treats this synthetic FP register as though it +Unfortunately GDB still treats this synthetic FP register as though it is real. That in turn really confuses users (arm and ``print $fp'' VS ``info registers fp''). The synthetic FP should be separated out of the true register set presented to the user. @@ -956,10 +956,10 @@ How I would like the register file to work is more like: | map random cache bytes to target - dependant i-face + dependent i-face /|\ | - target dependant + target dependent such as [gG] packet or ptrace buffer @@ -990,7 +990,7 @@ The main objectives being: o a mechanism that clearly separates the gdb internal register cache from any - target (not architecture) dependant + target (not architecture) dependent specifics such as [gG] packets. Of course, like anything, it sounds good in theory. In reality, it @@ -1075,7 +1075,7 @@ Fix target_signal_from_host() etc. The name is wrong for starters. ``target_signal'' should probably be ``gdb_signal''. ``from_host'' should be ``from_target_signal''. -After that it needs to be multi-arched and made independant of any +After that it needs to be multi-arched and made independent of any host signal numbering. -- @@ -1222,7 +1222,7 @@ The serial code already does this. -- -Make MI interface accessable from existing CLI. +Make MI interface accessible from existing CLI. -- @@ -1337,7 +1337,7 @@ What should happen is that ``target xxxx'' should call a generic open an asynchronous target that may need to perform background tasks as part of the ``attach'' phase. -Unfortunatly, due to limitations in the old/creaking command.h +Unfortunately, due to limitations in the old/creaking command.h interface, that isn't possible. The function being called isn't told of the ``xxx'' or any other context information. diff --git a/gdb/command.c b/gdb/command.c index 56fb85e..4fc9aa1 100644 --- a/gdb/command.c +++ b/gdb/command.c @@ -2045,7 +2045,7 @@ _initialize_command (void) With no arguments, run an inferior shell."); /* NOTE: cagney/2000-03-20: Being able to enter ``(gdb) !ls'' would - be a really useful feature. Unfortunatly, the below wont do + be a really useful feature. Unfortunately, the below wont do this. Instead it adds support for the form ``(gdb) ! ls'' (i.e. the space is required). If the ``!'' command below is added the complains about no ``!'' command would be replaced by diff --git a/gdb/config/nm-lynx.h b/gdb/config/nm-lynx.h index 4a6f6e4..bdb1dfb 100644 --- a/gdb/config/nm-lynx.h +++ b/gdb/config/nm-lynx.h @@ -73,7 +73,7 @@ extern int child_wait (int pid, struct target_waitstatus *status); /* Lynx needs a special definition of this so that we can - print out the pid and thread number seperatly. */ + print out the pid and thread number seperately. */ /* override child_pid_to_str in inftarg.c */ diff --git a/gdb/corelow.c b/gdb/corelow.c index 4c7ac9f..148f5f4 100644 --- a/gdb/corelow.c +++ b/gdb/corelow.c @@ -329,7 +329,7 @@ core_open (char *filename, int from_tty) siggy = bfd_core_file_failing_signal (core_bfd); if (siggy > 0) /* NOTE: target_signal_from_host() converts a target signal value - into gdb's internal signal value. Unfortunatly gdb's internal + into gdb's internal signal value. Unfortunately gdb's internal value is called ``target_signal'' and this function got the name ..._from_host(). */ printf_filtered ("Program terminated with signal %d, %s.\n", siggy, diff --git a/gdb/main.c b/gdb/main.c index c6b9516..9ed2e2a 100644 --- a/gdb/main.c +++ b/gdb/main.c @@ -105,7 +105,7 @@ captured_command_loop (void *data) /* FIXME: cagney/1999-11-05: A correct command_loop() implementaton would clean things up (restoring the cleanup chain) to the state they were just prior to the call. Technically, this means that - the do_cleanups() below is redundant. Unfortunatly, many FUNC's + the do_cleanups() below is redundant. Unfortunately, many FUNCs are not that well behaved. do_cleanups should either be replaced with a do_cleanups call (to cover the problem) or an assertion check to detect bad FUNCs code. */ diff --git a/gdb/mips-tdep.c b/gdb/mips-tdep.c index 20d5446..cf48ced 100644 --- a/gdb/mips-tdep.c +++ b/gdb/mips-tdep.c @@ -819,7 +819,7 @@ mips32_next_pc (CORE_ADDR pc) } /* mips32_next_pc */ /* Decoding the next place to set a breakpoint is irregular for the - mips 16 variant, but fortunatly, there fewer instructions. We have to cope + mips 16 variant, but fortunately, there fewer instructions. We have to cope ith extensions for 16 bit instructions and a pair of actual 32 bit instructions. We dont want to set a single step instruction on the extend instruction either. diff --git a/gdb/remote.c b/gdb/remote.c index ef2f8b4..61a730e 100644 --- a/gdb/remote.c +++ b/gdb/remote.c @@ -2192,7 +2192,7 @@ serial device is attached to the remote system\n\ /* FIXME: cagney/1999-09-23: During the initial connection it is assumed that the target is already ready and able to respond to - requests. Unfortunatly remote_start_remote() eventually calls + requests. Unfortunately remote_start_remote() eventually calls wait_for_inferior() with no timeout. wait_forever_enabled_p gets around this. Eventually a mechanism that allows wait_for_inferior() to expect/get timeouts will be diff --git a/gdb/serial.h b/gdb/serial.h index 4aff1b0..0fea8ea 100644 --- a/gdb/serial.h +++ b/gdb/serial.h @@ -62,7 +62,7 @@ extern void serial_un_fdopen (serial_t scb); enum serial_rc { SERIAL_ERROR = -1, /* General error. */ SERIAL_TIMEOUT = -2, /* Timeout or data-not-ready during read. - Unfortunatly, through ui_loop_hook(), this + Unfortunately, through ui_loop_hook(), this can also be a QUIT indication. */ SERIAL_EOF = -3 /* General end-of-file or remote target connection closed, indication. Includes diff --git a/gdb/top.c b/gdb/top.c index ccb6aec..813a5ba4 100644 --- a/gdb/top.c +++ b/gdb/top.c @@ -564,7 +564,7 @@ return_to_top_level (enum return_reason reason) code also randomly used a SET_TOP_LEVEL macro that directly initialize the longjmp buffers. */ -/* MAYBE: cagney/1999-11-05: Should the catch_erros and cleanups code +/* MAYBE: cagney/1999-11-05: Should the catch_errors and cleanups code be consolidated into a single file instead of being distributed between utils.c and top.c? */ @@ -607,9 +607,9 @@ catch_errors (catch_errors_ftype *func, PTR args, char *errstring, val = (*func) (args); catch_return = saved_catch; - /* FIXME: cagney/1999-11-05: A correct FUNC implementaton will + /* FIXME: cagney/1999-11-05: A correct FUNC implementation will clean things up (restoring the cleanup chain) to the state they - were just prior to the call. Unfortunatly, many FUNC's are not + were just prior to the call. Unfortunately, many FUNC's are not that well behaved. This could be fixed by adding either a do_cleanups call (to cover the problem) or an assertion check to detect bad FUNCs code. */ @@ -672,7 +672,7 @@ do_captured_command (void *data) /* FIXME: cagney/1999-11-07: Technically this do_cleanups() call isn't needed. Instead an assertion check could be made that simply confirmed that the called function correctly cleaned up - after its self. Unfortunatly, old code (prior to 1999-11-04) in + after itself. Unfortunately, old code (prior to 1999-11-04) in main.c was calling SET_TOP_LEVEL(), calling the command function, and then *always* calling do_cleanups(). For the moment we remain ``bug compatible'' with that old code.. */ diff --git a/gdb/utils.c b/gdb/utils.c index e3e88df..0046e09 100644 --- a/gdb/utils.c +++ b/gdb/utils.c @@ -607,7 +607,7 @@ verror (const char *string, va_list args) char *err_string; struct cleanup *err_string_cleanup; /* FIXME: cagney/1999-11-10: All error calls should come here. - Unfortunatly some code uses the sequence: error_begin(); print + Unfortunately some code uses the sequence: error_begin(); print error message; return_to_top_level. That code should be flushed. */ error_begin (); -- 2.7.4